-
Notifications
You must be signed in to change notification settings - Fork 497
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
9488 internationalization issues #9493
9488 internationalization issues #9493
Conversation
@JayanthyChengan thanks for the pull request! These seem like straightforward changes. Standard "move English into properties files" stuff. I gave this PR a 3 in term of effort to review (which I've basically done) and QA. |
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.
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.
FWIW: #9451 has a couple of the same fixes.
@JayanthyChengan hi! Can you please resolve the merge conflicts in this pull request? They're in this file:
|
Like @qqmyers said this PR is probably the cause: We merged it a couple hours ago. |
…zationIssues # Conflicts: # src/main/webapp/file-download-button-fragment.xhtml
@pdurbin - resolved the conflicts |
@JayanthyChengan thank you! And I see @qqmyers already moved this to QA. Great! The only failing test... ... will be fixed by this PR: |
@JayanthyChengan When I clicked signup, I got a 500 error in the ui and this server.log error: |
@kcondon - Committed the correction |
@JayanthyChengan Thanks! I'm ready to merge it. |
Which issue(s) this PR closes: