Skip to content

i18n(fr): update integrations-reference & adapter-reference#13448

Merged
ArmandPhilippot merged 10 commits into
withastro:mainfrom
ArmandPhilippot:i18n/fr-update-integrations-reference
Mar 26, 2026
Merged

i18n(fr): update integrations-reference & adapter-reference#13448
ArmandPhilippot merged 10 commits into
withastro:mainfrom
ArmandPhilippot:i18n/fr-update-integrations-reference

Conversation

@ArmandPhilippot
Copy link
Copy Markdown
Member

@ArmandPhilippot ArmandPhilippot commented Mar 13, 2026

Description (required)

Related issues & labels (optional)

  • Suggested label: i18n

@github-actions github-actions Bot added the i18n Anything to do with internationalization & translation efforts - ask @YanThomas for help! label Mar 13, 2026
@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 13, 2026

Deploy Preview for astro-docs-2 ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 706a8ff
🔍 Latest deploy log https://app.netlify.com/projects/astro-docs-2/deploys/69c56ce4942e2300080d8746
😎 Deploy Preview https://deploy-preview-13448--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 project configuration.

@astrobot-houston
Copy link
Copy Markdown
Contributor

astrobot-houston commented Mar 13, 2026

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

File Note
fr/guides/integrations-guide/netlify.mdx Localization changed, will be marked as complete.
fr/guides/integrations-guide/vercel.mdx Localization changed, will be marked as complete.
fr/guides/upgrade-to/v5.mdx Localization changed, will be marked as complete.
fr/reference/adapter-reference.mdx Localization changed, will be marked as complete.
fr/reference/integrations-reference.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 changed the title i18n(fr): update integrations-reference.mdx i18n(fr): update integrations-reference & adapter-reference Mar 13, 2026
@ArmandPhilippot ArmandPhilippot force-pushed the i18n/fr-update-integrations-reference branch from 61ba67e to 809c2b5 Compare March 24, 2026 11:31
@ArmandPhilippot ArmandPhilippot marked this pull request as ready for review March 24, 2026 11:36
Copy link
Copy Markdown
Contributor

@ld-web ld-web left a comment

Choose a reason for hiding this comment

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

Just a few things, LGTM !

Comment thread src/content/docs/fr/reference/adapter-reference.mdx Outdated
Comment thread src/content/docs/fr/reference/adapter-reference.mdx Outdated
Détermine à quelle étape du cycle de vie de la page le middleware est exécuté, et comment le code du middleware est émis dans le résultat de la compilation.

Lorsque cette option est activée, elle empêche le code middleware d'être regroupé et importé par toutes les pages pendant la compilation :
Le mode ` classic` correspond au comportement par défaut d'Astro. Avec les pages pré-rendues, le middleware est exécuté lors de la compilation, et n'est pas réexécuté lorsque la page est demandée. Avec les pages dynamiques, le middleware est exécuté uniquement lors de la requête. Le code du middleware fait partie de votre paquet compilé pour le serveur.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
Le mode ` classic` correspond au comportement par défaut d'Astro. Avec les pages pré-rendues, le middleware est exécuté lors de la compilation, et n'est pas réexécuté lorsque la page est demandée. Avec les pages dynamiques, le middleware est exécuté uniquement lors de la requête. Le code du middleware fait partie de votre paquet compilé pour le serveur.
Le mode ` classic` correspond au comportement par défaut d'Astro. Avec les pages pré-rendues, le middleware est exécuté lors de la compilation, et n'est pas réexécuté lorsque la page est requêtée. Avec les pages dynamiques, le middleware est exécuté uniquement lors de la requête. Le code du middleware fait partie de votre paquet compilé pour le serveur.

I feel like "requêtée" is more precise, can be ignored

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I don't know for this one. This sounds a bit odd to me. This is valid according to the Wiktionary, but not according to the Larousse). And I don't think this is commonly used? I mean the error message for "the requested page is unavailable" is usually translated with "la page demandée est indisponible".

So, I think I'll ignore this one.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Yeah I understand. My point of view is that for developers reading the docs it would be more "semantically" correct, more "web development"-related. On a personal side I would always use "requêter une page" rather than "demander une page", but yeah, I understand it can be ignored :)

Comment thread src/content/docs/fr/reference/integrations-reference.mdx Outdated
Co-authored-by: ld-web <46715342+ld-web@users.noreply.github.com>
@ArmandPhilippot
Copy link
Copy Markdown
Member Author

Thanks for the review!

Some pages, like /integrations-guide/netlify, have been updated in #13571
I feel it will be easier to deal with the changes in another PR... But this means some of the files will be set as up-to-date in Lunaria while they are not.

So, I'll wait a bit before merging this one to see what is the best course of action here.

@ArmandPhilippot
Copy link
Copy Markdown
Member Author

We had a new change with the 6.1 release (#13586), so I pushed 7de8cd4. This is pretty small! When you have the chance, can you check if you're okay with it @ld-web? Then, I think we can finally merge this one! Thanks again for you help!

Copy link
Copy Markdown
Contributor

@ld-web ld-web left a comment

Choose a reason for hiding this comment

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

LGTM! You're welcome!

@ArmandPhilippot ArmandPhilippot merged commit f7d838f into withastro:main Mar 26, 2026
10 checks passed
@ArmandPhilippot ArmandPhilippot deleted the i18n/fr-update-integrations-reference branch March 26, 2026 17:41
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.

3 participants