Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

CRDCDH-80 Implement Unsaved Changes Prompt #3

Merged
merged 4 commits into from
Jun 15, 2023
Merged

CRDCDH-80 Implement Unsaved Changes Prompt #3

merged 4 commits into from
Jun 15, 2023

Conversation

amattu2
Copy link
Member

@amattu2 amattu2 commented Jun 7, 2023

Overview

This PR covers the implementation of an "Unsaved Changes" prompt that invokes user action when leaving the current form section with unsaved changed.

Notably, when leaving a section with unsaved changes the following options become available to the user:

  • Cancel – Cancel current action, hides prompt and stops invoked navigation action
  • Save – Save the current form section then navigate onward
  • Discard – Discards current changes and continues navigation

Note: Any changes restored (i.e. Changing the Firstname field then restoring it back to the original value) will not count toward unsaved changes.

image

@amattu2 amattu2 added the 🚧 Do Not Merge This PR is not ready for merging label Jun 7, 2023
@amattu2

This comment was marked as resolved.

@amattu2

This comment was marked as resolved.

@amattu2 amattu2 requested a review from jonkiky June 12, 2023 17:22
@amattu2 amattu2 marked this pull request as ready for review June 12, 2023 17:22
@amattu2 amattu2 removed the 🚧 Do Not Merge This PR is not ready for merging label Jun 12, 2023
@amattu2 amattu2 changed the base branch from dev to CRDCDH-33 June 12, 2023 19:40
@amattu2 amattu2 added the 🚧 Do Not Merge This PR is not ready for merging label Jun 13, 2023
@amattu2
Copy link
Member Author

amattu2 commented Jun 13, 2023

Pending font change.

@amattu2 amattu2 removed the 🚧 Do Not Merge This PR is not ready for merging label Jun 13, 2023
Base automatically changed from CRDCDH-33 to dev June 15, 2023 14:51
Copy link
Collaborator

@jonkiky jonkiky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@jonkiky jonkiky merged commit 8e660d0 into dev Jun 15, 2023
@jonkiky jonkiky deleted the CRDCDH-80 branch June 15, 2023 14:51
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants