Umbrello UML Modeller is a Unified Modelling Language (UML) diagram program based on KDE Technology.
UML allows you to create diagrams of software and other systems in a standard format to document or design the structure of your programs.
Umbrello UML Modeller is a Unified Modelling Language (UML) diagram program based on KDE Technology.
UML allows you to create diagrams of software and other systems in a standard format to document or design the structure of your programs.
https://invent.kde.org/kde/umbrello/-/merge_requests/4 is a followup of this request
I have everything configured correctly now and have completed the merge
request.
PS: phabricator is not good at managing reviews that consist of multiple patches. A better way to do this is to create a fork from the project https://invent.kde.org/kde/umbrello, push the patches into a separate branch and create a merge request.
See https://invent.kde.org/kde/umbrello/-/merge_requests/3 for an example.
In D27584#617299, @jhayes wrote:The requested changes have been made,
Please update the patch with the changes applied
The requested changes have been made, I will run a diff and double check then submit the changes on 2/25.
Thanks.
Here you go, and thanks you for your patience.
The last bunch of minor issues found while translating this. Thanks in advance for fixing them and for your patience.
Apologize for the typos.
Thanks in advance for fixing these minor typos. Sorry for overlooking them before.
Thanks. But again, please
Fix xml typos
See the update revision D27584
In D27575#616222, @jhayes wrote:It doesn't appear to be an option for me.
It doesn't appear to be an option for me.
And one more typo.
Please select "abandon ..." in the add action drop down list.
Thanks for all of the comments, I will get the minor fixes wraped up today.
There is also a line for specifying the version with which this extension request was added for the news and changelog (e.g https://umbrello.kde.org/changelog.php?20.04
And two more fixes.
Thanks in advance for fixing these minor issues.
Thanks for fixing the i18n() issue.
I can't comment on the code, but the bug number should not be in the first line of the commit message.
All tooltips should be in i18n() to be translatable, not in QLatin1String and not in tr().
Sure, I will look at those, I do have a fair amount of time.
In D27201#607195, @jhayes wrote:I am rooting around now for what to work on next.
I am rooting around now for what to work on next.
I'm going to apply this with a cleanup I found on testing this patch. Thanks for contributing
Looks good, except for the missing file and the mentioned issues, which needs an update. Since this commit affects translations, and the stable branch is locked for changes, this commit would have to go to the master branch.
There is some progress in https://invent.kde.org/carlschwan/umbrello-kde-org :)
In D26419#588155, @okellogg wrote:I followed the steps you described and created a merge request:
https://invent.kde.org/kde/umbrello/merge_requests/1
I merged this request into git master branch of kde/umbrello, thank you for your contribution.
I followed the steps you described and created a merge request:
https://invent.kde.org/kde/umbrello/merge_requests/1
If you do not know how to create a merge request, you must do so:
Please commit this patch yourself, otherwise I will be listed as author, as the unpatched arc does not use the original author or create a merge request.
BTW: there is a new way to submit code changes to umbrello by creating gitlab merge requests at https://invent.kde.org/kde/umbrello/merge_requests