8000 Fix : translation french "emails.recovery.body" by pomarec · Pull Request #4783 · appwrite/appwrite · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Fix : translation french "emails.recovery.body" #4783

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

Merged
merged 3 commits into from
Dec 22, 2022

Conversation

pomarec
Copy link
Contributor
@pomarec pomarec commented Nov 25, 2022

Wrong reference to project variable

What does this PR do?

Fixes translation french "emails.recovery.body"

Test Plan

Send an email recovery in french

Related PRs and Issues

#4782

Have you added your change to the Changelog?

yes

Have you read the Contributing Guidelines on issues?

yes

@christyjacob4
Copy link
Member

@pomarec thanks a lot for the contribution! Can you please resolve the conflicts in the changelog? Could you also give another look at the French translations to ensure everything looks good ?

@pomarec pomarec force-pushed the fix-4782-email-body-fr-translation branch from 923133c to 810e43f Compare November 26, 2022 12:02
@pomarec pomarec force-pushed the fix-4782-email-body-fr-translation branch from 810e43f to 31089b7 Compare November 26, 2022 12:04
@pomarec
Copy link
Contributor Author
pomarec commented Nov 26, 2022

I found another error, emails.certificate.body contained auto translated var names

FYI : emails.invitation.subject contains "%s" two times, not named vars. Its the same in every language.

Copy link
Contributor
@stnguyen90 stnguyen90 left a comment

Choose a reason for hiding this comment

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

Hey 👋 awesome work on your PR! We've approved your work and it'll be merged soon!

@eldadfux eldadfux merged commit 054ebc6 into appwrite:master Dec 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
0