Skip to content

Comments

i18n(fr): add upgrade-to/vX.mdx guides#8772

Merged
yanthomasdev merged 12 commits intowithastro:mainfrom
ArmandPhilippot:i18n/fr-add-guides/upgrade-to/vX.mdx
Jul 19, 2024
Merged

i18n(fr): add upgrade-to/vX.mdx guides#8772
yanthomasdev merged 12 commits intowithastro:mainfrom
ArmandPhilippot:i18n/fr-add-guides/upgrade-to/vX.mdx

Conversation

@ArmandPhilippot
Copy link
Member

@ArmandPhilippot ArmandPhilippot commented Jul 11, 2024

Description (required)

Related issues & labels (optional)

  • Suggested label: i18n

Additional notes about translation

  • I added a PR comment in upgrade-to/v3.mdx about shims to find out whether you think this word should be translated or not.

  • I had doubts on how to translate Breaking changes in the different guides, so I leave my reasoning here:

    • Modifications majeures could be right but we loose the "breaking" aspect
    • Electronjs, for example, uses Changements majeurs avec rupture de compatibilité but it seems too verbose to repeat it in the sentences
    • I chose to mostly use Changements cassants/Modifications cassantes and sometimes majeurs to avoid repeats

In both cases, feel free to leave a suggestion if you have a better idea!

@vercel

This comment was marked as outdated.

@netlify
Copy link

netlify bot commented Jul 11, 2024

Deploy Preview for astro-docs-2 ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 3b1ecd0
🔍 Latest deploy log https://app.netlify.com/sites/astro-docs-2/deploys/669a84b5cb6b610007ea20bd
😎 Deploy Preview https://deploy-preview-8772--astro-docs-2.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 site configuration.

@github-actions github-actions bot added the i18n Anything to do with internationalization & translation efforts - ask @YanThomas for help! label Jul 11, 2024
@ArmandPhilippot ArmandPhilippot force-pushed the i18n/fr-add-guides/upgrade-to/vX.mdx branch from 103c537 to f2aa44a Compare July 11, 2024 09:52
@astrobot-houston
Copy link
Contributor

astrobot-houston commented Jul 11, 2024

Lunaria Status Overview

🌕 This pull request will trigger status changes.

Learn more

By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.

You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale File Note
fr guides/upgrade-to/v1.mdx Localization added, will be marked as complete.
fr guides/upgrade-to/v2.mdx Localization added, will be marked as complete.
fr guides/upgrade-to/v3.mdx Localization added, will be marked as complete.
fr guides/upgrade-to/v4.mdx Localization added, will be marked as complete.
fr reference/errors/invalid-get-static-paths-entry.mdx Localization changed, will be marked as complete.
Warnings reference
Icon Description
🔄️ The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

@ArmandPhilippot ArmandPhilippot marked this pull request as ready for review July 13, 2024 17:05
ArmandPhilippot added a commit to ArmandPhilippot/astro-docs that referenced this pull request Jul 17, 2024
* Remove all mentions of jsconfig in French guides and references
(see withastro#8818)

Note: `src/content/docs/en/guides/upgrade-to/v1.mdx` is not included
since it does not exist yet. I'll update the translation in withastro#8772.
Copy link
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, good work!

Co-authored-by: Thomas Bonnet <thomasbnt@protonmail.com>
yanthomasdev added a commit that referenced this pull request Jul 19, 2024
* i18n(fr): remove jsconfig mentions in guides and references

* Remove all mentions of jsconfig in French guides and references
(see #8818)

Note: `src/content/docs/en/guides/upgrade-to/v1.mdx` is not included
since it does not exist yet. I'll update the translation in #8772.

* i18n(fr): add changes from minor release docs

Add changes only for `reference/cli-reference.mdx`.
See #8796

---------

Co-authored-by: Yan <61414485+yanthomasdev@users.noreply.github.com>
Copy link
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!

Copy link
Member

@yanthomasdev yanthomasdev left a comment

Choose a reason for hiding this comment

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

Damn, this is a big one, LGTM! I hope y'all are ready for the V5 upgrade guide 😆

@yanthomasdev yanthomasdev merged commit 4e6c947 into withastro:main Jul 19, 2024
@ArmandPhilippot ArmandPhilippot deleted the i18n/fr-add-guides/upgrade-to/vX.mdx branch July 19, 2024 15:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants