Skip to content

move english strings to bundle.properties#5341

Merged
kcondon merged 44 commits intoIQSS:developfrom
scholarsportal:dataverse-remove-english-strings
Dec 3, 2018
Merged

move english strings to bundle.properties#5341
kcondon merged 44 commits intoIQSS:developfrom
scholarsportal:dataverse-remove-english-strings

Conversation

@JayanthyChengan
Copy link
Contributor

mhvezina and others added 30 commits October 17, 2018 12:29
Translation of English terms found in bundle.properties from latest
"develop" code. Fixes for new and modified variables.
Minor fixes for various variables now that I can better see them in
context (that is by toggling between interfaces),
for permission.roleWas
Again, a few more fixes to the French interface now that this is seen in
context.
Additional translations in French for all new and/or modified English
terms/variables for all Commits between ~Oct 12th 2018 to Nov 8th 2018.
(https://github.com/scholarsportal/dataverse/commits/4856_FrenchPropertyFiles/src/main/java/Bundle.properties)
@coveralls
Copy link

coveralls commented Nov 26, 2018

Coverage Status

Coverage increased (+0.004%) to 17.703% when pulling d4b5457 on scholarsportal:dataverse-remove-english-strings into 3275681 on IQSS:develop.

Copy link
Contributor

@mheppler mheppler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Approved. Moving to QA.

@mheppler mheppler removed their assignment Nov 27, 2018
@kcondon kcondon self-assigned this Nov 30, 2018
@kcondon
Copy link
Contributor

kcondon commented Nov 30, 2018

@JayanthyChengan Can you please resolve the branch conflicts? I've already reviewed/tested this so it is ready to be merged. Thanks! This can wait until next week as I'm going home shortly, have a nice weekend!

@JayanthyChengan
Copy link
Contributor Author

@kcondon , resolved the conflicts and merged latest "develop" branch.

@kcondon
Copy link
Contributor

kcondon commented Dec 3, 2018

@JayanthyChengan Thanks! Merging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants