Skip to content

Improve date handling#22

Merged
thehale merged 1 commit intomainfrom
dev
Oct 22, 2021
Merged

Improve date handling#22
thehale merged 1 commit intomainfrom
dev

Conversation

@thehale
Copy link
Owner

@thehale thehale commented Oct 22, 2021

This project was processing dates without proper consideration of
their timezones. As such, several bizarre errors errors were
happening, especially around the start and end of a sprint.

For example, issues closed at 6pm Arizona time on the last day of
the sprint were not having their points credited to that day on
the burndown chart because GitHub returned that closed date as
1am UTC the following day.

This project now processes all dates internally using UTC. The
sprint_start_date and sprint_end_date are assumed to be
at 00:00:00 local time, and the final burndown chart is
rendered in local time.

These changes fix #17 and fix #21

This project was processing dates without proper consideration of
their timezones. As such, several bizarre errors errors were
happening, especially around the start and end of a sprint.

For example, issues closed at 6pm Arizona time on the last day of
the sprint were not having their points credited to that day on
the burndown chart because GitHub returned that closed date as
1am UTC the following day.

This project now processes all dates internally using UTC. The
`sprint_start_date` and `sprint_end_date` are assumed to be
at 00:00:00 *local time*, and the final burndown chart is
rendered in local time.

These changes fix #17 and fix #21
@thehale thehale merged commit efd02e8 into main Oct 22, 2021
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.

[bug] Issues closed at the end of a sprint don't get marked as completed [bug] KeyError: 'DATE' when DATE is outside of the sprint start/end date

1 participant