Bug 31149 - Use dayjs to parse dates
Summary: Use dayjs to parse dates
Status: Needs Signoff
Alias: None
Product: Koha
Classification: Unclassified
Component: Architecture, internals, and plumbing (show other bugs)
Version: unspecified
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Jonathan Druart
QA Contact: Testopia
URL:
Keywords:
Depends on: 30310
Blocks:
  Show dependency treegraph
 
Reported: 2022-07-13 07:04 UTC by Jonathan Druart
Modified: 2022-07-13 07:08 UTC (History)
3 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Bug 31149: Use dayjs to parse dates (3.67 KB, patch)
2022-07-13 07:08 UTC, Jonathan Druart
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Jonathan Druart 2022-07-13 07:04:42 UTC
We don't need our own JS functions to parse date, we can rely on dayjs for that.
Comment 1 Jonathan Druart 2022-07-13 07:08:18 UTC
Created attachment 137680 [details] [review]
Bug 31149: Use dayjs to parse dates

We can use the isValid's dayjs function instead of our own parsing
functions.

Test plan:
Confirm that you cannot enter an invalid date when using a date picker widget.