Skip to content

Improve French translation #1803

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

Merged
merged 1 commit into from
May 31, 2025
Merged

Improve French translation #1803

merged 1 commit into from
May 31, 2025

Conversation

PositroniumJS
Copy link
Contributor

A few improvements that were still missing

@stevencohn stevencohn merged commit 9f7948d into stevencohn:main May 31, 2025
@PositroniumJS
Copy link
Contributor Author

Thank you @stevencohn for the merges!
How to update Resources.fr-FR.resx now to take into account the changes?

@stevencohn
Copy link
Owner

stevencohn commented Jun 1, 2025

No need to update the resx file. It uses both files by looking at the hints file first and, if it doesn't find an entry, looks in the resx file.

The resx file is auto-generated using a combination of online translators. It needs to stay that way. The hints file stands in as the manual override.

Thanks for your help!

@PositroniumJS
Copy link
Contributor Author

No need to update the resx file. It uses both files by looking at the hints file first and, if it doesn't find an entry, looks in the resx file.

The resx file is auto-generated using a combination of online translators. It needs to stay that way. The hints file stands in as the manual override.

Thanks for your help!

I see, then what else could be wrong? After updating, even removing and installing again, I still see the old translations such as "d'accord" and "Un de plus" :(

@stevencohn
Copy link
Owner

I merged the PR after I released the latest build. Sorry, it will be included in the installer in the next release.

@PositroniumJS
Copy link
Contributor Author

I merged the PR after I released the latest build. Sorry, it will be included in the installer in the next release.

But the PR #1800 is included right?

@PositroniumJS
Copy link
Contributor Author

With the last update everything works perfectly, thank you very much!

weissm pushed a commit to weissm/OneMore that referenced this pull request Jul 12, 2025
weissm pushed a commit to weissm/OneMore that referenced this pull request Jul 12, 2025
weissm pushed a commit to weissm/OneMore that referenced this pull request Jul 12, 2025
weissm pushed a commit to weissm/OneMore that referenced this pull request Jul 12, 2025
weissm pushed a commit to weissm/OneMore that referenced this pull request Jul 12, 2025
weissm pushed a commit to weissm/OneMore that referenced this pull request Jul 12, 2025
weissm pushed a commit to weissm/OneMore that referenced this pull request Jul 12, 2025
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.

2 participants