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

[locale] correct translation of locale for Vietnam #4874

Merged
merged 3 commits into from
Mar 11, 2019

Conversation

chienkira
Copy link
Contributor

No description provided.

@jsf-clabot
Copy link

jsf-clabot commented Nov 25, 2018

CLA assistant check
All committers have signed the CLA.

@chienkira chienkira changed the title correct translation of locale for Vietnam locale: correct translation of locale for Vietnam Nov 25, 2018
@chienkira chienkira changed the title locale: correct translation of locale for Vietnam [locale] correct translation of locale for Vietnam Nov 25, 2018
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.05%) to 88.089% when pulling e0d93dd on chienkira:develop into c68b4f1 on moment:develop.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.05%) to 88.089% when pulling e0d93dd on chienkira:develop into c68b4f1 on moment:develop.

@coveralls
Copy link

coveralls commented Nov 25, 2018

Coverage Status

Coverage increased (+0.07%) to 88.41% when pulling c9afebf on chienkira:develop into 8e6c9b9 on moment:develop.

@marwahaha
Copy link
Member

@bangnk approval?

Please edit src/locale/vi.js.

@chienkira
Copy link
Contributor Author

@marwahaha Thank you for checking my PR. The check "Generated files — Don't edit moment.js, min/, locale/, build/*." didn't get passed. What else i need to do?

@marwahaha
Copy link
Member

  1. Please undo your change to locale/vi.js.
  2. Please make the change in src/locale/vi.js.
  3. Please change the tests in src/test/locale/vi.js.

Hope this helps!

@chienkira
Copy link
Contributor Author

@marwahaha Thank you very much for the details. I have updated PR.

@marwahaha
Copy link
Member

@chienkira can you re-do the last git commit with the same email address? You can set the email address like git config --global user.email marwahaha@berkeley.edu

@bangnk approval?

@chienkira
Copy link
Contributor Author

@marwahaha sorry my bad! I have re-done the commit with github email address. Thank you very much.

@chienkira
Copy link
Contributor Author

@marwahaha Sorry, any update? Is there any reviewer else?
@bangnk Hi, Are you still active on github?

@kylekatarnls
Copy link

Google Translate:
rồi = already
trước = before
tuần rồi lúc = last week
tuần trước lúc = the week before
Reverting translation:
last week = tuần trước

https://glosbe.com/en/vi/last%20week
trước: 24 occurences in articles
rồi: 4 occurences in articles

Both seems right but trước seems more common.

rồi seems to be the contrary of tới when trước would be the contrary of sau, But for this one, tới is more common:
https://glosbe.com/en/vi/next%20week

I was thinking of changing this for Carbon too. But I'm hesitating because of that.

@chienkira Are you a native Vietnamese speaker? Are you sure rồi can be changed to trước while keeping tới unchanged? (They can appear in the same sentence).

@chienkira
Copy link
Contributor Author

@kylekatarnls Thank you for checking. Actually I'm a Vietnamese.

Are you a native Vietnamese speaker? Are you sure rồi can be changed to trước while keeping tới unchanged?

Yes I'm sure. We say tuần trước for last week, not say tuần rồi in common. There is nothing likes tuần rồi in our language - Vietnamese.
About next week, we say both tuần sau and tuần tới, so IMP it's not necessary to make a change with tới.

@chienkira
Copy link
Contributor Author

@kylekatarnls Google translate is doing well. You can check it here.
https://translate.google.com/#view=home&op=translate&sl=en&tl=vi&text=last%20week

@kylekatarnls
Copy link

Seems also coherent with the current past : '%s trước',

As for what is worth, I'm convinced.

kylekatarnls added a commit to kylekatarnls/Carbon that referenced this pull request Feb 18, 2019
@chienkira
Copy link
Contributor Author

@marwahaha Is there any update? pls

@marwahaha
Copy link
Member

@chienkira This is fine. Would you like to be another locale author? If so, please add your name in the same format underneath @bangnk .

@chienkira
Copy link
Contributor Author

@marwahaha Thank you for checking my PR. Yes I'm very glad to be a locale author.

If so, please add your name in the same format underneath @bangnk

Sorry but I don't know what should I do exactly?

@marwahaha
Copy link
Member

Please add this line underneath line 3 of src/locale/vi.js:

//! author : Chien Kira : https://github.com/chienkira	

@chienkira
Copy link
Contributor Author

Thank you. I have edited that line.

@marwahaha marwahaha merged commit d0f0dd8 into moment:develop Mar 11, 2019
GmaMontero added a commit to despegar/moment that referenced this pull request Sep 4, 2019
* [misc] release process fixes (moment#4961)

* Fix locale abbreviation in comment

* Update package-lock.json

* [misc] Fix type of WeekSpec (moment#4634)

* Fix "the the" typo in test (moment#4578)

* [locale] Fix moment#4568: Insert punjabi for "next" (moment#4583)

* Fix type of WeekSpec

You can set `dow` without setting `doy`, so the type should reflect that

* [new locale] (oc-lnc) Occitan (moment#4957)

* Add oc-lnc files

* Fix calendar test

* Fix the rest of tests

* Add locale author

* Dont make changes to package-lock

* Add newline at end of file

* [bugfix] Fix moment#4962 updateLocale with null to reset (moment#4966)

* [locale] es: Add es translation for invalidDate (moment#4560)

* Add es translation for invalidDate

* Fix lint

* [locale] improve mk translation (moment#4958)

* Update mk.js

* Update mk.js

* Update mk.js

* Update mk.js

* [locale] improve mk translation

* [tests] Fix various text typos (moment#4976)

* [locale] mr (Marathi) (moment#4990)

* tests according to changes in mr.js

* [locale] mr (Marathi)

We Marathi people follow standards mentioned below, to label timespans.

Label | Time Span 
------------ | -------------
पहाटे (post midnight, pre-sunrise) |    >= 12am to < 6am 
सकाळी (post sunrise, pre-noon) |    >= 6am to < 12pm
दुपारी (afternoon) |    >= 12pm to < 5pm
सायंकाळी (evening) |    >= 5pm to < 8pm
रात्री (night) |    >= 8pm to < 12am

Actually, 12am is confusing in Marathi tradition. In modern clock system, Moment 12am is considered in arriving date as 0am. And in Marathi tradition it is referred to as final moment of the previous date (Sort of, 0:01 is starting minute of the arriving date) . To comply with modern clock system, I propose to refer to 0am as पहाटे , so that it is considered as moment in the arriving date. Hope this discription itself is not confusing :)

Another point is about range of पहाटे, ie. >= 0am to < 6am. Though conventionally, upto 2 am, we refer to the time as रात्री or पहाटे, both alternatively, I want to specifically call पहाटे for early times in the arriving date and रात्री for late times in the departing date, to make it more specific and well-defined, avoiding confusion about date of the time.

* [locale] correct translation of locale for Vietnam (moment#4874)

* correct translation of locale for Vietnam

* correct translation of locale for Vietnam (edit proper src files)

* Update author comment line

* [locale] (en-sg) Rename locale en-SG to en-sg (moment#5025)

* [locale] typos in Irish locale for months (moment#5127)

Fixed some typos in the Irish locale regarding months.

* [locale] (br) Breton move from 12-hour to 24-hour notation (moment#4975)

* Breton : move from 12-hour to 24-hour notation moment#4974 moment#4974

* Replace [e] by :

* [locale] bn: Improve translation as per Bangla Academy (moment#5055)

* Delete unused locales

* Remove unused locales

* Update test and build

* Add untracked locale
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.

None yet

5 participants