-
-
Notifications
You must be signed in to change notification settings - Fork 414
docs: improve translation visibility and lower barrier to entry #7034
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
base: main
Are you sure you want to change the base?
docs: improve translation visibility and lower barrier to entry #7034
Conversation
Add Weblate badge, dedicated Translations section in CONTRIBUTING.md, and rewrite locale/README.md to be beginner-friendly. Addresses OSGeo#7006
0a95f6b to
e11fc32
Compare
|
Apologies for the inconvenience @echoix . I accidentally included commits from another branch (I guess during the compare and pull request). I have force-pushed to clean it up, and the PR now only contains the relevant documentation changes. |
|
I didn't post any review comments yet, so it really doesn't matter. I didn't have time to read either |
echoix
left a comment
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.
Overall I like the tone, it is positive and convincing, almost like an advertisement ;)
The readme in locale/ folder still contains the needed older info, and updated, except for some few points which were not true anymore before this change, and needs to be true now
If someone else wants to read after that to see how it fits with our other guidelines, the greater picture, I'd appreciate, but once the little points addressed, I plan on having this merged anyways, it's really close
locale/README.md
Outdated
| @@ -1,24 +1,36 @@ | |||
| # HOWTO translate GRASS GIS messages | |||
| # Contributing to GRASS GIS Translations | |||
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.
| # Contributing to GRASS GIS Translations | |
| # Contributing to GRASS Translations |
In 8.5, and since somewhere in spring, GRASS GIS was renamed, or rebranded as GRASS; it isn't only a GIS
locale/README.md
Outdated
| # Contributing to GRASS GIS Translations | ||
|
|
||
| ## Where to translate user message | ||
| **Welcome!** Thank you for your interest in helping translate GRASS GIS. |
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.
| **Welcome!** Thank you for your interest in helping translate GRASS GIS. | |
| **Welcome!** Thank you for your interest in helping translate GRASS. |
locale/README.md
Outdated
| - Email the development mailing list at `grass-dev@lists.osgeo.org` | ||
| ([subscribe here](https://lists.osgeo.org/mailman/listinfo/grass-dev)) |
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.
The discourse subsection grass developers replaced the grass-dev mailing list. All the content was migrated to there.
locale/README.md
Outdated
| ### Quick start | ||
|
|
||
| **Please help us translate all the GRASS messages!!** | ||
| 1. **Create an account** at [OSGeo Weblate](https://weblate.osgeo.org/) |
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.
Validate with your experience on if it is enough or the correct steps. When going in private navigation, I don't see a way to create the account on the Weblate sign in page, on my phone anyways.
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.
Adding the additional one step of making OSGeo User ID first
CONTRIBUTING.md
Outdated
|
|
||
| Translations are automatically submitted to the GRASS repository as pull requests. | ||
| For help or questions, join the | ||
| [GRASS development community on Discourse](https://discourse.osgeo.org/c/grass/developer/61) |
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.
@petrasovaa wasn't there an old translation mailing list? Should we migrate it too? Just thinking about it, as we're linking to the grass-dev list equivalent on discourse.
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.
Oh, good catch, it even has posts from 2025!
https://lists.osgeo.org/pipermail/grass-translations/
The trend was to replace these with user or dev discussions, but I one can make an argument for a separate list.
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.
Hm, I need to checks with PSC again, but I think we are moving away from it, so I wouldn't add it here.
|
I'm glad you liked the tone @echoix , so should I commit the suggestions you made , or should I wait for others to give their opinions or anything you want me to do ... |
|
The ones you know, make them, but maybe there will be nothing to change after that |
CONTRIBUTING.md
Outdated
| You can also report issues, plan new features, | ||
| or explore <https://grass.osgeo.org/get-involved/>. | ||
|
|
||
| ## Translations |
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 would move this after AI policy. And I would shorten it, it should mainly point to the locale/README. So e.g. you don't need here getting started and the "why translate" seems excessive (and too obviously AI-generated).
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.
Sure, I had asked for suggestions for the same but I do get it from your perspective , doing the changes ...
This PR addresses #7006 by improving documentation to encourage translation contributions.
Changes
Motivation
Translation activity has been minimal except for French and Swedish. These changes aim to:
Fixes #7006