Skip to content
This repository was archived by the owner on Jan 16, 2022. It is now read-only.

feat: i18n for UI via crowdin translations#198

Closed
juanpicado wants to merge 2 commits intomasterfrom
feat-i18n-integration
Closed

feat: i18n for UI via crowdin translations#198
juanpicado wants to merge 2 commits intomasterfrom
feat-i18n-integration

Conversation

@juanpicado
Copy link
Copy Markdown
Member

Type: feat

Description:

  • Download strings from crowdin
  • Prove integration with react-i18next

Resolves #102

@juanpicado juanpicado added the wip label Oct 13, 2019
@codecov
Copy link
Copy Markdown

codecov bot commented Oct 13, 2019

Codecov Report

Merging #198 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #198   +/-   ##
=======================================
  Coverage   88.73%   88.73%           
=======================================
  Files         117      117           
  Lines         932      932           
  Branches      168      147   -21     
=======================================
  Hits          827      827           
- Misses         89       92    +3     
+ Partials       16       13    -3
Impacted Files Coverage Δ
src/utils/styles/media.ts 81.81% <0%> (ø) ⬆️
src/components/DetailSidebar/DetailSidebar.tsx 85% <0%> (ø) ⬆️

@priscilawebdev
Copy link
Copy Markdown
Contributor

@juanpicado I am looking forward to have i18n in our code base! 💪👏 Kudos!!

"react-dom": "16.10.0",
"react-emotion": "9.2.12",
"react-hot-loader": "4.12.11",
"react-i18next": "10.13.1",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please why did you decided to use "react-i18next"?.. it's just matter of curiosity..I have never used this library before.

@priscilawebdev priscilawebdev added the enhancement New feature or request label Jan 2, 2020
@priscilawebdev
Copy link
Copy Markdown
Contributor

closing, because it was already merged in this PR: #432

@priscilawebdev priscilawebdev deleted the feat-i18n-integration branch March 8, 2020 16:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement New feature or request wip

Projects

None yet

Development

Successfully merging this pull request may close these issues.

i18n at UI

2 participants