-
Notifications
You must be signed in to change notification settings - Fork 2
Feature/date time range picker #394
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
base: master
Are you sure you want to change the base?
Conversation
|
@rs-mgb Hey, last commit 6 month ago. Is this still valid? |
|
@cod3cow This is probably still valid and, as mentioned in the description, has some features to validate the date inputs. However, merging this branch would require a significant amount of retesting to ensure that the current validation functions are working as expected. Since we currently don’t need the features included in this branch, I would suggest ignoring and/or removing this PR. |
Thanks. I will discuss this also with other devs. |
Wrong branch name; these changes are related to date validation. If you are interested, I can give a short presentation about it during the next Pair Programming session.