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

dayjs 2.0 - add plugins 'LocalizedFormat' and 'LocaleData' - based on pr 2050 and pr 2139 #2140

Open
wants to merge 34 commits into
base: next
Choose a base branch
from

Conversation

BePo65
Copy link
Contributor

@BePo65 BePo65 commented Nov 25, 2022

Add plugin LocalizedFormat.
Add plugin LocaleData.

The tests require locale es. Therefore this pr is based on pr #2139 (and on pr #2050).

BePo65 added 30 commits August 6, 2022 18:12
Add tests from ArraySupport in dayjs v1.x
Prior to using 'vi.setSystemTime(now)' the mocking of the timers must be
activated with 'vi.useFakeTimers()'
Differentiating between 'date' and 'day' is not very easy.
… set

Units for add / sub / diff / get / set are modifications of the units for
parsing / formatting and are used in several places. Therefore type
definitions for these derived types are added.
BREAKING CHANGE: in dayjs 1.0: only days and weeks are rounded to the
nearest integer.
In moment: days and months are rounded to the nearest integer;
Weeks, quarters, and years are converted to days or months, and then
rounded to the nearest integer.
Negative numbers are rounded as positive numbers with the sign added to
the result.
An array with only a year value (first element) returned "invalid Date";
in this case the correct date is returned, only if the second parameter
is not undefined (but e.g. null)
Example for offset not full or halve hours is '1879-12-31' for
timezone 'Europe/Berlin', where utcOffset is '00:53'.
Solves issue iamkun#1996.
The type 'GetterFn' is needed by plugins that add getter / setter to Dayjs.
Change name of parameter with value to add / subtract from the quite
generic 'number' to 'value' to avoid get consistent with the parameter
names e.g. of 'set'.
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.

1 participant