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

Rely on luxon for our datetime calculations (refactor) #1235

Open
Skaiir opened this issue Aug 14, 2024 · 0 comments
Open

Rely on luxon for our datetime calculations (refactor) #1235

Skaiir opened this issue Aug 14, 2024 · 0 comments
Labels
backlog Queued in backlog spring cleaning Could be cleaned up one day

Comments

@Skaiir
Copy link
Contributor

Skaiir commented Aug 14, 2024

What should we do?

We should clean up date times to use luxon over the native js date objects, as they are poorly implemented and all the custom code we write is subject to a lot of issues.

Why should we do it?

To reduce our changes of having bugs like #1223, and simplify the code structure, making maintenance easier down the line.

@Skaiir Skaiir added backlog Queued in backlog spring cleaning Could be cleaned up one day labels Aug 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backlog Queued in backlog spring cleaning Could be cleaned up one day
Projects
None yet
Development

No branches or pull requests

1 participant