-
Notifications
You must be signed in to change notification settings - Fork 479
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
8346 License internationalizataion #9125
8346 License internationalizataion #9125
Conversation
…alizataion # Conflicts: # src/main/java/edu/harvard/iq/dataverse/dataset/DatasetUtil.java
…icenseInternationalizataion
…olarsportal/dataverse into licenseInternationalizataion
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. The one question I have is whether it would make sense to refactor to have a getLocalizesLicenseDetails(License, String) or (License, Enum) interface. For the former that would mean doing license.getName() once within the method instead of all the callers.
…olarsportal/dataverse into licenseInternationalizataion
@qqmyers - Please take a look at the changes when you have time. Thanks |
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 - thanks!
Gave it a 10. Should be just testing with French or whatever. |
Added to 12/1/2022 sprint on 12/5/2022 |
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 didn't test this but the code and docs look fine to me. All the API tests passed. Approving.
Which issue(s) this PR closes:
Closes #8346