-
Notifications
You must be signed in to change notification settings - Fork 1
MPDX-9241 Replace cancel with discard button #1585
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
Conversation
|
Preview branch generated at https://mpdx-9241-add-discard-button.d3dytjb8adxkk5.amplifyapp.com |
Bundle sizes [mpdx-react]Compared against 26b469f No significant changes found |
|
@dr-bizz Since a user is not allowed to edit when a request is |
dr-bizz
left a comment
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.
Approving this, but found some small things
src/components/Reports/Shared/CalculationReports/SubmitModal/SubmitModal.tsx
Outdated
Show resolved
Hide resolved
src/components/Reports/Shared/CalculationReports/ReceiptStep/Receipt.tsx
Outdated
Show resolved
Hide resolved
src/components/Reports/Shared/CalculationReports/PanelLayout/PanelLayout.tsx
Outdated
Show resolved
Hide resolved
3d94767 to
f7041f8
Compare
f7041f8 to
f2e685e
Compare
Description
Based on new figma designs, we must replace "Cancel" with "Discard" button and update the submit modal text.
Testing
Testing Edit page discard button (
In Progressstatus):/reports/housingAllowanceTesting New page discard button (
In Progressstatus):Testing cancel button (
In Progressstatus):Testing cancel button (
Pendingstatus)cancelledChecklist:
/pr-reviewcommand locally and fixed any relevant suggestions