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

"Short Weekday" (ddd) throws error when setting the value in the text input #2884

Closed
3 tasks done
AnatoliiPavlov opened this issue Dec 6, 2023 · 1 comment · Fixed by #2887
Closed
3 tasks done

Comments

@AnatoliiPavlov
Copy link

Prerequisites

Describe the issue

It can be reproduced even in the demo https://getdatepicker.com/6/plugins/customDateFormat.html
The stackblitz example provides the same behavior.

  1. Add 'ddd' to the beginning of format string so it will become 'ddd dd/MM/yyyy HH:mm'.
  2. Press 'Change' button. Sample picker will update its value to something like 'ter. 05/12/2023 21:10'.
  3. Add or subtract minutes (like 'ter. 05/12/2023 21:11') directly in the text input and press Enter
  4. Observe the Dev Tools console - it will display an error 'errors.ts:140 Uncaught Error: TD: Custom Date Format: Unable to parse provided input: ter. 05/12/2023 21:11, format: ddd dd/MM/yyyy HH:mm'
  5. Press picker icon - picker won't reflect your changes (it will display 21:10 instead of expected 21:11)

StackBlitz fork

https://stackblitz.com/edit/tempus-dominus-v6-simple-setup-x2v18u?file=index.js

What operating system(s) are you seeing the problem on?

Windows

What browser(s) are you seeing the problem on?

Chrome

What version of are you using? You can find this information from the sample StackBlitz.

6.7.16

What your browser's locale? You can find this information from the sample StackBlitz.

en-US

@AnatoliiPavlov AnatoliiPavlov added State: Triage Tickets that need to be triaged. Type: Bug labels Dec 6, 2023
@github-project-automation github-project-automation bot moved this to 🆕 New in Tempus Dominus Dec 6, 2023
@Eonasdan Eonasdan added Breaking Change and removed State: Triage Tickets that need to be triaged. labels Dec 18, 2023
@Eonasdan Eonasdan moved this from 🆕 New to 🏗 In progress in Tempus Dominus Dec 18, 2023
@Eonasdan Eonasdan added Area: TypeScript State: Pending Release This issue has been resolved in the dev branch and is waiting on a new release and removed Breaking Change labels Dec 20, 2023
Eonasdan added a commit that referenced this issue Dec 21, 2023
@Eonasdan Eonasdan mentioned this issue Dec 21, 2023
@github-project-automation github-project-automation bot moved this from 🏗 In progress to ✅ Done in Tempus Dominus Dec 21, 2023
Eonasdan added a commit that referenced this issue Dec 21, 2023
* fixes #2884

* fixes #2886

* fixes #2881

* fixes #2879

* fixes #2857

* #2793

* #2793
v6.9.4
@Eonasdan
Copy link
Owner

This is fixed in 6.9.4

@Eonasdan Eonasdan removed the State: Pending Release This issue has been resolved in the dev branch and is waiting on a new release label Dec 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants