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
chore(update): update cal lib and date adapter (DSP-1758) #309
Conversation
@gautschr Could you try to choose the BC date which had the wrong number of days in February ? |
@@ -262,8 +262,8 @@ describe('ValueService', () => { | |||
|
|||
const period = calDateJDN.toJDNPeriod(); | |||
|
|||
expect(period.periodStart).toEqual(1721060); | |||
expect(period.periodEnd).toEqual(1721060); | |||
expect(period.periodStart).toEqual(1721058); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@gautschr I had to adapt these expectations.
The Gregorian calendar date of January 1st 1 BCE used to be 1721060 in JDN, now it is 1721058.
Since the date is before 1582, it was calculated like a Julian calendar date, right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes!
@@ -262,8 +262,8 @@ describe('ValueService', () => { | |||
|
|||
const period = calDateJDN.toJDNPeriod(); | |||
|
|||
expect(period.periodStart).toEqual(1721060); | |||
expect(period.periodEnd).toEqual(1721060); | |||
expect(period.periodStart).toEqual(1721058); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes!
resolves DSP-1758