-
Notifications
You must be signed in to change notification settings - Fork 147
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
Persian language #1020
Persian language #1020
Conversation
Thanks for the pull request, @CodeWithEmad! Please note that it may take us up to several weeks or months to complete a review and merge your PR. Feel free to add as much of the following information to the ticket as you can:
All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here. Please let us know once your PR is ready for our review and all tests are green. |
Hi @CodeWithEmad! Just flagging that there are a couple failing checks. Thanks! |
4fa7e77
to
56b991e
Compare
Hi @mphilbrick211 |
@CodeWithEmad looks like there are some CI failures, can you have a look? |
59f8d9c
to
001d4bf
Compare
Translations have been added to this pull request as of August 7th, 2023.
001d4bf
to
ed32989
Compare
consider it done @e0d! |
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #1020 +/- ##
=======================================
Coverage 84.52% 84.52%
=======================================
Files 113 113
Lines 2190 2190
Branches 633 633
=======================================
Hits 1851 1851
Misses 323 323
Partials 16 16
☔ View full report in Codecov by Sentry. |
package-lock.json
Outdated
"version": "1.2.5", | ||
"resolved": "https://registry.npmjs.org/word-wrap/-/word-wrap-1.2.5.tgz", | ||
"integrity": "sha512-BN22B5eaMMI9UMtjrGd5g5eCYPpCPDUy0FJXbYsaT5zYxjFOckS53SQDE3pWkVoWpHXVb3BrYcEN4Twa55B5cA==", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this change intentional?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no, it wasn't. I can remove the commit if it's not acceptable :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, please.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
ed32989
to
2cb56e6
Compare
Hi @zainab-amir! When you get a moment, would you be able to approve the tests to run on this PR? They bounced back to needing approval. |
@CodeWithEmad 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future. |
Co-authored-by: Zainab Amir <zainab.amir@arbisoft.com>
This will add Persian (fa_IR) language to the Authn MFE.
Additionally, the translations for other languages have been updated, and a few typos have been corrected.