Skip to content
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

Update intro.md [Doc French translation] #2146

Open
wants to merge 2 commits into
base: trunk
Choose a base branch
from

Conversation

beryl-dlg
Copy link
Contributor

@beryl-dlg beryl-dlg commented Feb 12, 2025

Motivation for the change, related issues

Hello, following the path to inaugurate french translations to the Playground documentation
Initiated with @jennydupuy at the last WordCamp Bretagne Contribution Day 2025

Sorry if I didn't do it right, I'm not sure I should have made a PR but i'm not familiar with this process…

https://wordpress.github.io/wordpress-playground/contributing/translations/

Implementation details

Testing Instructions (or ideally a Blueprint)

@beryl-dlg beryl-dlg changed the title Update intro.md [French translation] Update intro.md [Doc French translation] Feb 12, 2025
@adamziel
Copy link
Collaborator

Thank you so much! It seems like there's one flaky failing CI check that we can ignore, and one actual issue reported by the runner — it detected a dead link. I'm on my phone and not sure if that's accurate — let me loop in @juanmaguitar who initially set up the language support

@juanmaguitar
Copy link
Collaborator

Thanks for this work @beryl-dlg. I have added a few comments.

Once you have applied these changes you can test them locally by going to /wordpress-playground/packages/docs/site and from that folder run

npm run dev -- --locale fr

PS: The command above requires that previously the dependencies of the project had been installed by doing npm install from /wordpress-playground

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Inbox
Development

Successfully merging this pull request may close these issues.

3 participants