Skip to content
This repository was archived by the owner on Apr 26, 2024. It is now read-only.

Update configuration file with relevant I18n data#2644

Merged
ovflowd merged 5 commits into
nodejs:mainfrom
Kunal0007:i18nconfig
Aug 18, 2022
Merged

Update configuration file with relevant I18n data#2644
ovflowd merged 5 commits into
nodejs:mainfrom
Kunal0007:i18nconfig

Conversation

@Kunal0007

Copy link
Copy Markdown
Contributor

Check List

  • I have read the Contributing Guidelines and made commit messages that follow the guideline.
  • I have run npm run lint:js -- --fix and/or npm run lint:md -- --fix for my JavaScript and/or Markdown changes.
    • This is important as most of the cases your code changes might not be correctly linted
  • I have run npm run test to check if all tests are passing, and/or npm run test -- -u to update snapshots if I created and/or updated React Components.
  • I have checked that the build works locally and that npm run build work fine.
  • I've covered new added functionality with unit tests if necessary.

Description

Updated src/i18n/configs.json file with correct metadata from all the languages.

Updated data missing from all these languages, such as their date formats, if those languages are RTL (Right-to-left) or LTR (Left-to-right), and their hreflangs.

Reference:
hreflang:- https://hreflang.org/list-of-hreflang-codes/
Date format and langDir:- https://meta.wikimedia.org/wiki/Date_formats_in_various_languages

Related Issues

Fixes #2619

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

Merging #2644 (583aed9) into main (9d4b2cc) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #2644   +/-   ##
=======================================
  Coverage   84.33%   84.33%           
=======================================
  Files         100      100           
  Lines        1117     1117           
  Branches      313      313           
=======================================
  Hits          942      942           
  Misses        169      169           
  Partials        6        6           

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

@ovflowd ovflowd left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

That's great! Thank you so much for the contribution, Kunal! 💖

@AugustinMauroy

Copy link
Copy Markdown
Member

ref : #2639

@ovflowd ovflowd merged commit 70ee0cf into nodejs:main Aug 18, 2022
Comment thread src/i18n/config.json
"dateFormat": "DD.MM.YYYY",
"hrefLang": "",
"dateFormat": "YYYY.MM.DD",
"hrefLang": "zh-Hant",

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.

Sorry to interrupt, but I'm afraid there's a tiny mistake 😂

The t in zh-Hant means "traditional" while s in zh-Hans means "Simplified".

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.

I just wrote a possible fix, hoping this might help. 😄 main...zedifen:i18n-ch-fix

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Oh, gotcha! Could you open a PR? 🙈

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Sorry, I got a little confused 😅.

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.

Oh, gotcha! Could you open a PR? 🙈

Ok, I've opened a PR. 😃

Sorry, I got a little confused 😅.

Oh, it's alright 😆 There are lots of variants of Chinese, and their codes do look quite similar

@Kunal0007 Kunal0007 deleted the i18nconfig branch August 18, 2022 14:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update src/i18n/configs.json with relevant I18n data

6 participants