-
Notifications
You must be signed in to change notification settings - Fork 72
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature/ap 3948 #1072
Feature/ap 3948 #1072
Conversation
I probably need to run through some parts of this PR |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Goodness, there's a lot of work here.
I have a repetitive comment "Localize messagebox title" which would probably be most easily addressed by doing something similar to dashboard.util.ZKMessageCtrl
or org.apromore.portal.common.notification.Notification
and having some static utility methods.
...ore-Components/Apromore-Portal/src/main/java/org/apromore/portal/common/i18n/I18nConfig.java
Outdated
Show resolved
Hide resolved
...ore-Components/Apromore-Portal/src/main/java/org/apromore/portal/common/i18n/I18nConfig.java
Outdated
Show resolved
Hide resolved
...re-Components/Apromore-Portal/src/main/java/org/apromore/portal/common/i18n/I18nSession.java
Outdated
Show resolved
Hide resolved
...Apromore-Portal/src/main/java/org/apromore/portal/dialogController/BPMNEditorController.java
Outdated
Show resolved
Hide resolved
...promore-Portal/src/main/java/org/apromore/portal/dialogController/BaseListboxController.java
Show resolved
Hide resolved
...in-Portal-Plugin/src/main/java/org/apromore/plugin/portal/useradmin/UserAdminController.java
Outdated
Show resolved
Hide resolved
...in-Portal-Plugin/src/main/java/org/apromore/plugin/portal/useradmin/UserAdminController.java
Show resolved
Hide resolved
...in-Portal-Plugin/src/main/java/org/apromore/plugin/portal/useradmin/UserAdminController.java
Show resolved
Hide resolved
...-Admin-Portal-Plugin/src/main/java/org/apromore/plugin/portal/useradmin/UserAdminPlugin.java
Outdated
Show resolved
Hide resolved
...-Admin-Portal-Plugin/src/main/java/org/apromore/plugin/portal/useradmin/UserAdminPlugin.java
Outdated
Show resolved
Hide resolved
Reviewed and updated, except localization of Messagebox. Need to discuss how the static class defined in Core could pull resource bundle from various plugins. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm satisfied that my previous change requests and review comments have been considered. Good to merge!
No description provided.