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.

Sign up for GitHub

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

Jump to bottom

fix-l10n(fr): misleading sentence #21981

Merged
merged 1 commit into from
Jun 30, 2024
Merged

fix-l10n(fr): misleading sentence #21981

merged 1 commit into from
Jun 30, 2024

Conversation

flawyte
Copy link
Contributor

@flawyte flawyte commented Jun 27, 2024

Description

The sentence in french was misleading and incorrect. I had to switch to the english version to make sure I get it right.
@flawyte flawyte requested a review from a team as a code owner June 27, 2024 09:14
@flawyte flawyte requested review from tristantheb and removed request for a team June 27, 2024 09:14
@github-actions github-actions bot added the l10n-fr Issues related to French content. label Jun 27, 2024
@github-actions GitHub Actions
Copy link
Contributor

Preview URLs

Flaws (4)

URL: /fr/docs/Mozilla/Add-ons/WebExtensions/API/storage/StorageArea/get
Title: StorageArea.get()
Flaw count: 4

  • macros:
    • /fr/docs/Mozilla/Add-ons/WebExtensions/Background_scripts does not exist but fell back to /en-US/docs/Mozilla/Add-ons/WebExtensions/Background_scripts
    • /fr/docs/Mozilla/Add-ons/Contact_us does not exist but fell back to /en-US/docs/Mozilla/Add-ons/Contact_us
  • broken_links:
    • Can't resolve /fr/docs/Web/JavaScript/Reference/Objets_globaux/Promise
    • Can't resolve /fr/docs/Web/JavaScript/Reference/Objets_globaux/Promise
cw118
cw118 approved these changes Jun 30, 2024
View reviewed changes
Copy link
Member

@cw118 cw118 left a comment

Choose a reason for hiding this comment

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

Looks great, thank you for this contribution @flawyte !
@cw118 cw118 merged commit 6a73f62 into mdn:main Jun 30, 2024
6 checks passed
@flawyte flawyte deleted the patch-1 branch July 2, 2024 07:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
l10n-fr Issues related to French content.
2 participants
@flawyte @cw118