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

Add localization #602

Merged
merged 41 commits into from
Mar 4, 2020
Merged

Add localization #602

merged 41 commits into from
Mar 4, 2020

Conversation

Mavrin
Copy link
Collaborator

@Mavrin Mavrin commented Feb 17, 2020

This pull request introduces localization infrastructure. I localized all components.
Screen Shot 2020-02-18 at 00 12 55
Also I added some translation for ru lang.

frontend/.gitignore Outdated Show resolved Hide resolved
frontend/package.json Outdated Show resolved Hide resolved
@Mavrin Mavrin force-pushed the localization branch 3 times, most recently from 0b153d2 to b8e2520 Compare February 17, 2020 20:30
@Mavrin Mavrin marked this pull request as ready for review February 17, 2020 20:45
@Mavrin Mavrin requested a review from umputun as a code owner February 17, 2020 20:45
@Mavrin Mavrin requested a review from akellbl4 February 17, 2020 20:46
@Mavrin Mavrin force-pushed the localization branch 2 times, most recently from c510ab5 to c828714 Compare February 22, 2020 09:16
paskal
paskal previously approved these changes Feb 22, 2020
Copy link
Sponsor Collaborator

@paskal paskal left a comment

Choose a reason for hiding this comment

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

Untranslated part:
image
Also, I see that language change doesn't occur on page reload and only force-reload, but it's not a big problem from my point of view.

Otherwise, everything is good, great job!

@Mavrin Mavrin requested a review from paskal February 22, 2020 22:19
Copy link
Sponsor Collaborator

@paskal paskal left a comment

Choose a reason for hiding this comment

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

I think this should be merged together with Readme changes. I don't like to have functionality merged and having some parts of it (tests, or documentation as in this case) left for other time, as currently is the best time to add these changes.

@Mavrin Mavrin requested a review from paskal February 23, 2020 10:14
paskal
paskal previously approved these changes Feb 23, 2020
frontend/app/common/fetcher.ts Outdated Show resolved Hide resolved
frontend/app/utils/loadLocale.ts Show resolved Hide resolved
frontend/app/remark.tsx Outdated Show resolved Hide resolved
frontend/app/last-comments.tsx Outdated Show resolved Hide resolved
paskal
paskal previously approved these changes Feb 24, 2020
Copy link
Contributor

@codiflow codiflow left a comment

Choose a reason for hiding this comment

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

The name of the variable in line 60 (frontend/app/locales/de.json) should be reply instead of replay.

So this would need a change in the following files too:

  • frontend/app/components/comment-form/comment-form.tsx, Line 389 (Also change Replay to Reply)
  • frontend/app/locales/en.json
  • frontend/app/locales/ru.json

The German translation is already done - I would commit it once the pull request got merged.

@Mavrin Mavrin requested a review from akellbl4 March 3, 2020 20:46
frontend/package.json Outdated Show resolved Hide resolved
akellbl4
akellbl4 previously approved these changes Mar 3, 2020
Copy link
Collaborator

@akellbl4 akellbl4 left a comment

Choose a reason for hiding this comment

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

@Mavrin great job!

Co-Authored-By: Pavel Mineev <pavel@mineev.me>
@codiflow
Copy link
Contributor

codiflow commented Mar 4, 2020

So am I right that merging seems to be possible very soon? :-)

Can't wait to contribute my German translation.

@umputun umputun merged commit b22cd5f into umputun:master Mar 4, 2020
@umputun umputun added this to the v1.6 milestone Apr 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants