Skip to content

i18n(fr) Update french translation with last PRs#6683

Merged
dreyfus92 merged 20 commits into
withastro:mainfrom
McFlyPartages:up_fr_last
Jan 31, 2024
Merged

i18n(fr) Update french translation with last PRs#6683
dreyfus92 merged 20 commits into
withastro:mainfrom
McFlyPartages:up_fr_last

Conversation

@McFlyPartages
Copy link
Copy Markdown
Contributor

@McFlyPartages McFlyPartages commented Jan 28, 2024

Description (required)

Update french translation for multiple file outdated

@vercel
Copy link
Copy Markdown

vercel Bot commented Jan 28, 2024 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
docs ✅ Ready (Inspect) Visit Preview Jan 31, 2024 9:23pm

@github-actions github-actions Bot added the i18n Anything to do with internationalization & translation efforts - ask @YanThomas for help! label Jan 28, 2024
thomasbnt
thomasbnt requested changes Jan 28, 2024
View reviewed changes
Copy link
Copy Markdown
Member

@thomasbnt thomasbnt left a comment

Choose a reason for hiding this comment

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

Two suggestions about space in a note and a duplication of the word "plugin"

Comment thread src/content/docs/fr/install/manual.mdx Outdated
Comment thread src/content/docs/fr/guides/integrations-guide/alpinejs.mdx Outdated
McFlyPartages and others added 2 commits January 28, 2024 23:05
@McFlyPartages @thomasbnt
Update src/content/docs/fr/install/manual.mdx
f9247d5 
Co-authored-by: Thomas Bonnet <thomasbnt@protonmail.com>
@McFlyPartages @thomasbnt
Update src/content/docs/fr/guides/integrations-guide/alpinejs.mdx
ae33953 
Co-authored-by: Thomas Bonnet <thomasbnt@protonmail.com>
@McFlyPartages
Copy link
Copy Markdown
Contributor Author

McFlyPartages commented Jan 28, 2024

Two suggestions about space in a note and a duplication of the word "plugin"

Thanks it's ok now

thomasbnt
thomasbnt approved these changes Jan 28, 2024
View reviewed changes
Copy link
Copy Markdown
Member

@thomasbnt thomasbnt left a comment

Choose a reason for hiding this comment

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

LGTM!

VoxelMC
VoxelMC approved these changes Jan 29, 2024
View reviewed changes
Copy link
Copy Markdown
Contributor

@VoxelMC VoxelMC left a comment

Choose a reason for hiding this comment

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

Lots of work! Thank you very much @McFlyPartages :)

I added 3 suggestions to remove the DontEditWarning component, as the import is not needed in the translated versions.

Awesome work!!!

As a non-native speaker, I reviewed this PR using the Docs Translation Review Checklist and it LGTM!

Comment thread src/content/docs/fr/reference/errors/missing-index-for-internationalization.mdx Outdated
Comment thread src/content/docs/fr/reference/errors/i18n-not-enabled.mdx Outdated
Comment thread src/content/docs/fr/reference/errors/astro-glob-no-match.mdx Outdated
McFlyPartages and others added 3 commits January 29, 2024 19:33
@McFlyPartages @VoxelMC
Update src/content/docs/fr/reference/errors/missing-index-for-interna…
cfdae65 
…tionalization.mdx

Co-authored-by: voxel!() <voxelmc@hotmail.com>
@McFlyPartages @VoxelMC
Update src/content/docs/fr/reference/errors/i18n-not-enabled.mdx
04156d0 
Co-authored-by: voxel!() <voxelmc@hotmail.com>
@McFlyPartages @VoxelMC
Update src/content/docs/fr/reference/errors/astro-glob-no-match.mdx
471500f 
Co-authored-by: voxel!() <voxelmc@hotmail.com>
@McFlyPartages
Copy link
Copy Markdown
Contributor Author

McFlyPartages commented Jan 29, 2024

Lots of work! Thank you very much @McFlyPartages :)

I added 3 suggestions to remove the DontEditWarning component, as the import is not needed in the translated versions.

Awesome work!!!

As a non-native speaker, I reviewed this PR using the Docs Translation Review Checklist and it LGTM!

Thanks

dreyfus92
dreyfus92 approved these changes Jan 31, 2024
View reviewed changes
Copy link
Copy Markdown
Member

@dreyfus92 dreyfus92 left a comment

Choose a reason for hiding this comment

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

LGTM, merci à tous 😁

@dreyfus92 dreyfus92 merged commit c5326dd into withastro:main Jan 31, 2024
@McFlyPartages McFlyPartages deleted the up_fr_last branch February 1, 2024 06:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@thomasbnt thomasbnt thomasbnt approved these changes

@dreyfus92 dreyfus92 dreyfus92 approved these changes

+1 more reviewer

@VoxelMC VoxelMC VoxelMC approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

i18n Anything to do with internationalization & translation efforts - ask @YanThomas for help!

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@McFlyPartages @thomasbnt @VoxelMC @dreyfus92