Ramsundar - Resolved Cancel and Submit button functionality in Daily Equipment Logs#4554
Conversation
✅ Deploy Preview for highestgoodnetwork-dev ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
one-community
added
the
High Priority - Please Review First
HemanthNidamanuru
left a comment
HemanthNidamanuru
left a comment
There was a problem hiding this comment.
Hi Ramsundar,
The functionality is working fine but the dropdown and calendar are not aligned in dark mode.
…quipment_Log Resolve merge conflicts in EDailyActivityLog.jsx by combining both branches: kept Spinner + isSubmitting from the PR fix alongside validation states, confirmation dialog, and tooltips from development. Co-authored-by: Cursor <cursoragent@cursor.com>
|
Hey, This issue comes under my task. I have solved it and also cleared all the merge conflicts. Please review it. Thanks! 
|
HemanthNidamanuru
left a comment
HemanthNidamanuru
left a comment
There was a problem hiding this comment.
Hi Ramsundar,
I tested this PR locally. The dropdown and calendar alignment in dark mode is looking good now.
DeepighaJ
left a comment
DeepighaJ
left a comment
There was a problem hiding this comment.
- Checked out to current branch and validated PR locally and validated for multiple projects and Check-In and Check-out states in light mode and dark mode. Form validation and data submission works as expected.
- Reset works as expected.
- Success message and reset message appears on submission and cancel as expected.
|
Thank you all, merging! |
4 participants
Description
Related PRS (if any):
This frontend PR is related to the latest development backend branch.
Main changes explained:
How to test:
Screenshots or videos of changes:
https://www.loom.com/share/56c15f3e504149ea83cd50367b612f90
Note:
Include the information the reviewers need to know.