Skip to content

Conversation

@rs-mgb
Copy link
Collaborator

@rs-mgb rs-mgb commented Oct 30, 2024

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.

@cod3cow
Copy link
Collaborator

cod3cow commented May 12, 2025

@rs-mgb Hey, last commit 6 month ago. Is this still valid?

@rs-mgb
Copy link
Collaborator Author

rs-mgb commented May 12, 2025

@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.

@cod3cow
Copy link
Collaborator

cod3cow commented May 14, 2025

@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.

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.

3 participants