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: localized format, closes #1154 #1685

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

Akryum
Copy link

@Akryum Akryum commented Nov 8, 2021

Closes #1154
Closes #1383
Closes #1499

@codecov
Copy link

codecov bot commented Nov 8, 2021

Codecov Report

Merging #1685 (307b798) into dev (b5a1391) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##               dev     #1685   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          179       179           
  Lines         1996      1997    +1     
  Branches       507       508    +1     
=========================================
+ Hits          1996      1997    +1     
Impacted Files Coverage Δ
src/plugin/timezone/index.js 100.00% <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 b5a1391...307b798. Read the comment docs.

@VincentGarreau
Copy link

Could be great to merge it :)

Here's a demo that shows the issue compared to other lib like date-fns or the official Intl API: https://codesandbox.io/s/timezones-abbr-js-dates-lib-ko60p?file=/src/Dayjs.vue

@lk77
Copy link

lk77 commented Dec 28, 2021

i have the same issue, i hope this gets merged soon

@VincentGarreau
Copy link

Any news regarding this fix?

@vronifuchsberger
Copy link

Any news on this?

@bilalbutt044
Copy link

this is still not fixed. still not getting the timezone name in abbreviation.

@kanhaiya-mishra
Copy link

@iamkun Can you please check if this can be merged. Will help a lot of users. Thanks

@alessandro308
Copy link

Any news on it?

@MariusBaci
Copy link

Any news on this ?

@itseramin
Copy link

Please merge @iamkun 🧡

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Correct output of timezone ("z") for CET/CEST Abbreviated Timezone for IST/GMT+5:30
9 participants