-
Notifications
You must be signed in to change notification settings - Fork 348
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
Don't allow future dates for facility triage #4094
Conversation
✅ Deploy Preview for care-egov-staging ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
test approved |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- can you change all the mui components in this form to our custom components
- and also add constraints in the DateInput component to disable future dates (something like maxDate)
@khavinshankar Done! |
LGTM |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Pranshu1902 also change the date picker too, not only the buttons, we already have custom date pickers
@Pranshu1902 @khavinshankar is it good for testing |
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
LGTM |
|
* dont validate future dates * removed mui and added max date contraint * added custom date picker * improved ui
Fixes #4093
http://localhost:4000/facility/072e9364-a108-42e2-a0a5-3e70f9d90e3c/triage
The form raises an error when submitting the form with an invalid date