-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
feat: date time picker improvements #5297
base: develop
Are you sure you want to change the base?
Conversation
Build Error! No Linked Issue found. Please link an issue or mention it in the body using #<issue_id> |
Build Error! No Linked Issue found. Please link an issue or mention it in the body using #<issue_id> |
874afe2
to
092cf69
Compare
138a4ae
to
907e044
Compare
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.
Two minor issue https://www.loom.com/share/6d79a063386748e180938c7d4320424f
@srikanthccv addressed the comments. |
Functionally LGTM. This is a good change but not an urgent one so please add tests. |
yes will merge this post adding the tests. |
Summary
1 week/s , minute/s , hour/s, day/s
in the date time picker inputRelated Issues / PR's
fixes #4321
fixes #4322
Screenshots
NA
Affected Areas and Manually Tested Areas