-
Notifications
You must be signed in to change notification settings - Fork 501
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
4685 ui toggle for language selection #4789
4685 ui toggle for language selection #4789
Conversation
Latest French translation - Consolidation
…685-UI-toggle-for-language-selection # Conflicts: # src/main/java/Bundle_fr.properties
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.
Looks good
@JayanthyChengan I have not had a chance today to review the UI for this, but hope to get to this as my next task. |
…ggle-for-language-selection
|
||
} | ||
|
||
private static String readUrl(String urlString) throws Exception { |
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.
could any tests be added for this?
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 tested the workflow only with logger messages.
Closed as superseded by the other PR for this. |
That other PR being #4841. |
Submitting this PR in reference to the issue: #4685
@pdurbin
@mheppler
added by @pdurbin to associate the pull request with the issue in Waffle
connects to #4685