Skip to content

Add French translation - #23

Merged
LittleChest merged 8 commits into
LeavesMC:mainfrom
ouafax:i18n/#22/add-french-translation
Oct 22, 2025
Merged

Add French translation#23
LittleChest merged 8 commits into
LeavesMC:mainfrom
ouafax:i18n/#22/add-french-translation

Conversation

@ouafax

@ouafax ouafax commented Oct 22, 2025

Copy link
Copy Markdown
Contributor

Linked to issue #22
Add french translation to the website.

@netlify

netlify Bot commented Oct 22, 2025

Copy link
Copy Markdown

Deploy Preview for leavesmc ready!

Name Link
🔨 Latest commit d081edf
🔍 Latest deploy log https://app.netlify.com/projects/leavesmc/deploys/68f8f65f38c83d0008a63314
😎 Deploy Preview https://deploy-preview-23--leavesmc.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@LittleChest
LittleChest self-requested a review October 22, 2025 14:42
@LittleChest

Copy link
Copy Markdown
Member

Ok, one last step!

You need to add this language to i18n and language selectors.
I can help you with this if you need help.

@ouafax

ouafax commented Oct 22, 2025

Copy link
Copy Markdown
Contributor Author

Ok, one last step!

You need to add this language to i18n and language selectors. I can help you with this if you need help.

Can you confirm I need to copy the filter and add it under

.filter((language) => language !== "en")

and
.filter((language) => language !== "en")

If that's good I can push the updated file

@LittleChest LittleChest left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

a small flaw

Comment thread src/i18n/fr.json Outdated
fix: typo in software name

Co-authored-by: LittleChest <gtuffy123456@gmail.com>
@LittleChest

Copy link
Copy Markdown
Member

Ok, one last step!
You need to add this language to i18n and language selectors. I can help you with this if you need help.

Can you confirm I need to copy the filter and add it under

.filter((language) => language !== "en")

and

.filter((language) => language !== "en")

If that's good I can push the updated file

maybe, to be honest, I forget why I did this, probably to exclude fallback languages

.filter((language) => language !== "fr") 

and i18n.ts
you can check this commit
i think it should be called fr-FR

@ouafax
ouafax requested a review from LittleChest October 22, 2025 15:10
@ouafax

ouafax commented Oct 22, 2025

Copy link
Copy Markdown
Contributor Author

Ok, one last step!
You need to add this language to i18n and language selectors. I can help you with this if you need help.

Can you confirm I need to copy the filter and add it under

.filter((language) => language !== "en")

and

.filter((language) => language !== "en")

If that's good I can push the updated file

maybe, to be honest, I forget why I did this, probably to exclude fallback languages

.filter((language) => language !== "fr") 

and i18n.ts you can check this commit i think it should be called fr-FR

Updated french translation file name, added it to i18n.ts and in the navbar

@LittleChest LittleChest left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you for your contribution!

@LittleChest
LittleChest merged commit d7ce811 into LeavesMC:main Oct 22, 2025
4 checks passed
@ouafax
ouafax deleted the i18n/#22/add-french-translation branch October 22, 2025 15:32
@LittleChest LittleChest linked an issue Oct 22, 2025 that may be closed by this pull request
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.

[i18n] Add French to website language

2 participants