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

fix: Locale: bo [Tibetan]: corrected the orders in formats #1823

Merged
merged 3 commits into from
Apr 14, 2022

Conversation

KalDhondup
Copy link
Contributor

Hi, dayjs is great. I have used it in my recent project. Thanks. But there is some formats which are incorrect in Tibetan.
like, In Tibetan its, ལོ་ 6 (means 6 years) rather than 6 ལོ ( which sounds like years 6 in English ).

Here are some correct formats that we are able to do. Thanks again! Cheers! :)

Hi, dayjs is great. I have used it in my recent project. Thanks. But there is some formats which are incorrect in Tibetan. 
like, In Tibetan its, ལོ་ 6 (means 6 years) rather than 6 ལོ ( which sounds like years 6 in English ). 

Here are some correct formats that we are able to do. Thanks again! Cheers! :)
@KalDhondup KalDhondup changed the title Locale: bo [Tibetan]: correcting some false orders in the formats fix: Locale: bo [Tibetan]: corrected the orders in formats Mar 8, 2022
@iamkun
Copy link
Owner

iamkun commented Mar 11, 2022

LGTM. @KalDhondup would you please ask a native speaker to review this PR before it gets merged? Thanks

fixes after review from a native speaker
@codecov
Copy link

codecov bot commented Mar 17, 2022

Codecov Report

Merging #1823 (787ef50) into dev (dcbf170) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               dev     #1823   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          180       180           
  Lines         2034      2034           
  Branches       524       524           
=========================================
  Hits          2034      2034           
Impacted Files Coverage Δ
src/locale/bo.js 100.00% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dcbf170...787ef50. Read the comment docs.

Copy link
Contributor Author

@KalDhondup KalDhondup left a comment

Choose a reason for hiding this comment

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

Fixes from a native Tibetan speaker

@KalDhondup
Copy link
Contributor Author

Hi @iamkun, Thanks. We discussed with a native speaker and he did some more corrections. It's thoroughly reviewed.

KalDhondup added a commit to KalDhondup/dayjs that referenced this pull request Mar 22, 2022
Hi @iamkun, These changes are reviewed with help Tibetan native speaker. There is some issue with previous pull request(iamkun#1823) and we are making a new pull request.  Thanks :)
@KalDhondup KalDhondup mentioned this pull request Mar 22, 2022
@iamkun
Copy link
Owner

iamkun commented Mar 27, 2022

Hi, please fix the lint error before the PR get merged

/home/travis/build/iamkun/dayjs/src/locale/bo.js
  11:1  error  Trailing spaces not allowed  no-trailing-spaces

fixing lint error.
@KalDhondup
Copy link
Contributor Author

Hi @iamkun, The issue is fixed. Its good to go 👍

@iamkun iamkun merged commit e790516 into iamkun:dev Apr 14, 2022
iamkun pushed a commit that referenced this pull request Apr 15, 2022
## [1.11.1](v1.11.0...v1.11.1) (2022-04-15)

### Bug Fixes

* add Bengali (Bangladesh) [bn-bd] locale ([#1806](#1806)) ([840ed76](840ed76))
* refactor replace deprecated String.prototype.substr() ([#1836](#1836)) ([627fa39](627fa39))
* Update German [de] locale, adds the abbreviations for month including a . in the end, as in September -> Sept. ([#1831](#1831)) ([4e2802c](4e2802c))
* update Italian (Switzerland) [it-ch] locale relativeTime ([#1829](#1829)) ([8e6d11d](8e6d11d))
* update Kurdish [ku] locale strings and formatted output contains non-standard kurdish characters ([#1848](#1848)) ([a597d0b](a597d0b))
* update locale bo [Tibetan]: corrected the orders in formats ([#1823](#1823)) ([e790516](e790516))
@iamkun
Copy link
Owner

iamkun commented Apr 15, 2022

🎉 This PR is included in version 1.11.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

andrewhood125ruhuc added a commit to andrewhood125ruhuc/SidRH2 that referenced this pull request May 10, 2022
## [1.11.1](iamkun/dayjs@v1.11.0...v1.11.1) (2022-04-15)

### Bug Fixes

* add Bengali (Bangladesh) [bn-bd] locale ([#1806](iamkun/dayjs#1806)) ([840ed76](iamkun/dayjs@840ed76))
* refactor replace deprecated String.prototype.substr() ([#1836](iamkun/dayjs#1836)) ([627fa39](iamkun/dayjs@627fa39))
* Update German [de] locale, adds the abbreviations for month including a . in the end, as in September -> Sept. ([#1831](iamkun/dayjs#1831)) ([4e2802c](iamkun/dayjs@4e2802c))
* update Italian (Switzerland) [it-ch] locale relativeTime ([#1829](iamkun/dayjs#1829)) ([8e6d11d](iamkun/dayjs@8e6d11d))
* update Kurdish [ku] locale strings and formatted output contains non-standard kurdish characters ([#1848](iamkun/dayjs#1848)) ([a597d0b](iamkun/dayjs@a597d0b))
* update locale bo [Tibetan]: corrected the orders in formats ([#1823](iamkun/dayjs#1823)) ([e790516](iamkun/dayjs@e790516))
andrewhood125ruhuc added a commit to andrewhood125ruhuc/SidRH2 that referenced this pull request May 10, 2022
## [1.11.1](iamkun/dayjs@v1.11.0...v1.11.1) (2022-04-15)

### Bug Fixes

* add Bengali (Bangladesh) [bn-bd] locale ([#1806](iamkun/dayjs#1806)) ([840ed76](iamkun/dayjs@840ed76))
* refactor replace deprecated String.prototype.substr() ([#1836](iamkun/dayjs#1836)) ([627fa39](iamkun/dayjs@627fa39))
* Update German [de] locale, adds the abbreviations for month including a . in the end, as in September -> Sept. ([#1831](iamkun/dayjs#1831)) ([4e2802c](iamkun/dayjs@4e2802c))
* update Italian (Switzerland) [it-ch] locale relativeTime ([#1829](iamkun/dayjs#1829)) ([8e6d11d](iamkun/dayjs@8e6d11d))
* update Kurdish [ku] locale strings and formatted output contains non-standard kurdish characters ([#1848](iamkun/dayjs#1848)) ([a597d0b](iamkun/dayjs@a597d0b))
* update locale bo [Tibetan]: corrected the orders in formats ([#1823](iamkun/dayjs#1823)) ([e790516](iamkun/dayjs@e790516))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants