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

Close #847: Translate Time To Leave to Hebrew #849

Merged
merged 17 commits into from
Oct 12, 2022
Merged

Close #847: Translate Time To Leave to Hebrew #849

merged 17 commits into from
Oct 12, 2022

Conversation

TomerPacific
Copy link
Contributor

Related issue

Closes #847

Context / Background

Following what was detailed in #475, I followed the guidelines and added a translation to Hebrew.

What change is being introduced by this PR?

  • I added the language locale in src/configs/app.config.js
  • I created a he folder under locals with the relevant files and translations
  • I added a Hebrew translated readme file
  • I added a link to this readme in the main readme file

How will this be tested?

I have looked at the markdown in a preview tool to see that it looks appropriately.


  • I confirm I'm a native or fluent speaker of the language I'm translating to.

Copy link
Collaborator

@tupaschoal tupaschoal left a comment

Choose a reason for hiding this comment

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

Seems mostly ready to go, just a few nitpicks.

README.md Outdated Show resolved Hide resolved
docs/README-he.md Outdated Show resolved Hide resolved
locales/he/translation.json Show resolved Hide resolved
@codecov
Copy link

codecov bot commented Oct 11, 2022

Codecov Report

Merging #849 (fe21e21) into main (17d6e44) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #849   +/-   ##
=======================================
  Coverage   59.42%   59.42%           
=======================================
  Files          26       26           
  Lines        2080     2080           
  Branches      305      305           
=======================================
  Hits         1236     1236           
  Misses        844      844           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

TomerPacific and others added 2 commits October 11, 2022 20:48
Removing all other readme links

Co-authored-by: Tulio Leao <tupaschoal@gmail.com>
@tupaschoal
Copy link
Collaborator

\changelog-update
Message: Translation: Time to Leave is now available in Hebrew (he)!

@tupaschoal tupaschoal changed the title Feature/Hebrew Translation Close #847: Translate Time To Leave to Hebrew Oct 12, 2022
@tupaschoal tupaschoal merged commit 5f7bb3e into thamara:main Oct 12, 2022
@tupaschoal
Copy link
Collaborator

Thanks @TomerPacific! If in the future we have new or changed strings in the app, can we mention you for help on keeping Hebrew up-to-date?

@TomerPacific
Copy link
Contributor Author

@tupaschoal - Of course. You can count on me.
Thank you for your help.

@TomerPacific TomerPacific deleted the feature/hebrew-translation branch October 12, 2022 16:22
@tupaschoal
Copy link
Collaborator

Cheers!

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.

Translate Time to Leave to Hebrew
3 participants