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

feat(datepicker): add Vietnam (vi) locale to date picker #5221

Merged
merged 2 commits into from
May 20, 2019

Conversation

nartc
Copy link
Contributor

@nartc nartc commented May 18, 2019

PR Checklist

Before creating new PR, please take a look at checklist below to make sure that you've done everything that needs to be done before we can merge it.

  • read and followed the CONTRIBUTING.md guide.
  • built and tested the changes locally.
  • added/updated tests.
  • added/updated API documentation.
  • added/updated demos.

nhutlbuit
nhutlbuit previously approved these changes May 18, 2019
@vtnhut94
Copy link

Add vn locale

@kuribohzz98
Copy link

Add vi locale 👍

@hieu-ht
Copy link

hieu-ht commented May 18, 2019

+1

@codecov
Copy link

codecov bot commented May 18, 2019

Codecov Report

Merging #5221 into development will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##           development    #5221   +/-   ##
============================================
  Coverage        51.51%   51.51%           
============================================
  Files                3        3           
  Lines               99       99           
  Branches            17       17           
============================================
  Hits                51       51           
  Misses              38       38           
  Partials            10       10

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 984ca74...5583136. Read the comment docs.

1 similar comment
@codecov
Copy link

codecov bot commented May 18, 2019

Codecov Report

Merging #5221 into development will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##           development    #5221   +/-   ##
============================================
  Coverage        51.51%   51.51%           
============================================
  Files                3        3           
  Lines               99       99           
  Branches            17       17           
============================================
  Hits                51       51           
  Misses              38       38           
  Partials            10       10

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 984ca74...5583136. Read the comment docs.

Copy link
Contributor

@envil envil left a comment

Choose a reason for hiding this comment

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

Correct translations

Copy link

@tnanhpt tnanhpt left a comment

Choose a reason for hiding this comment

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

good

@tiennampham23
Copy link

Helpful (y)

@vuonghv
Copy link

vuonghv commented May 20, 2019

Very helpful. Thanks

@tiendo93
Copy link

Thank you!!!

@ludmilanesvitiy
Copy link
Contributor

Datepicker tested, looks good.

@Domainv Domainv merged commit 4c1f2bf into valor-software:development May 20, 2019
@nartc nartc deleted the vi-locale branch May 20, 2019 12:45
@Domainv Domainv added this to the 4.3.0 milestone May 20, 2019
@quannv1198
Copy link

good

leo6104 pushed a commit to leo6104/ngx-bootstrap that referenced this pull request Oct 10, 2019
IraErshova pushed a commit to IraErshova/ngx-bootstrap that referenced this pull request Jan 20, 2020
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.