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

TAN-2973 Replace old date range picker #9907

Open
wants to merge 37 commits into
base: master
Choose a base branch
from

Conversation

luucvanderzee
Copy link
Contributor

@luucvanderzee luucvanderzee commented Dec 24, 2024

Changelog

Changed

  • Replaced old date range picker

Fixed

  • Fixed all 'one day off' errors that were happening with old date range picker

Copy link

New task

@cl-dev-bot
Copy link
Collaborator

cl-dev-bot commented Dec 24, 2024

Messages
📖 Changelog provided 🎉
📖 Notion issue: TAN-2973
📖

Run the e2e tests

📖 Check translation progress

Generated by 🚫 dangerJS against 827fc3e

@luucvanderzee luucvanderzee marked this pull request as ready for review December 24, 2024 18:46
// This function makes sure that the date is always interpreted as midnight in the user's timezone.
const backendDatestringRegex = /^\d{4}-\d{2}-\d{2}$/;

export const parseBackendDateString = (dateString?: string) => {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think I finally solved all the stupid 'one day off' errors. We just have to make sure to use this utility whenever we receive a string from the backend that has the format yyyy-MM-DD

@luucvanderzee
Copy link
Contributor Author

@EdwinKato functional test would be most important here!

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.

2 participants