Skip to content

Comments

#8818 - Added reference to translation tool to Internationalization section of the documentation#8819

Merged
kcondon merged 6 commits intoIQSS:developfrom
universidadeaveiro:8818_easyTranslationHelper
Aug 3, 2022
Merged

#8818 - Added reference to translation tool to Internationalization section of the documentation#8819
kcondon merged 6 commits intoIQSS:developfrom
universidadeaveiro:8818_easyTranslationHelper

Conversation

@j-n-c
Copy link
Contributor

@j-n-c j-n-c commented Jun 28, 2022

…ection of the documentation

What this PR does / why we need it:

Which issue(s) this PR closes:

Closes #8818

Special notes for your reviewer:

Suggestions on how to test this:

Does this PR introduce a user interface change? If mockups are available, please link/include them here:

Is there a release notes update needed for this change?:

Additional documentation:

Copy link
Member

@pdurbin pdurbin left a comment

Choose a reason for hiding this comment

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

The preview at https://dataverse-guide--8819.org.readthedocs.build/en/8819/installation/config.html#tools looks quite good but I added a couple comments.

@4tikhonov should https://weblate.dataverse.org be listed as well?

@4tikhonov
Copy link
Contributor

Hi @pdurbin, Weblate should be included. We're planning its upgrade soon, btw.

@pdurbin
Copy link
Member

pdurbin commented Jul 14, 2022

@4tikhonov ok, great. @j-n-c would you like to try adding it?

@j-n-c
Copy link
Contributor Author

j-n-c commented Jul 14, 2022

@pdurbin Sure!
@4tikhonov, who should be given credit for https://weblate.dataverse.org?

@4tikhonov
Copy link
Contributor

Hi @j-n-c, I have coordinated it in @SSHOC project.

@j-n-c
Copy link
Contributor Author

j-n-c commented Jul 15, 2022

@pdurbin , @4tikhonov I've updated the docs with the reference to Weblate. Please check if there is anything I could/should improve

@pdurbin
Copy link
Member

pdurbin commented Jul 15, 2022

https://dataverse-guide--8819.org.readthedocs.build/en/8819/installation/config.html#tools-for-translators look good to me. Thanks, @j-n-c!

@4tikhonov any comments?

(@j-n-c one tiny thing is that both bullets should end with a period or not. Either way is fine with me.)

@j-n-c
Copy link
Contributor Author

j-n-c commented Jul 15, 2022

@pdurbin , good eye! Added period where it was missing

@kcondon kcondon self-assigned this Aug 3, 2022
@kcondon kcondon merged commit 472311a into IQSS:develop Aug 3, 2022
@pdurbin pdurbin added this to the 5.12 milestone Aug 26, 2022
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.

Documentation Update: Include easyTranslationHelper in documentation

5 participants