Skip to content

Comments

Updated to create alert rather than modal for auto-recover [D: 831]#579

Merged
abeverley merged 3 commits intoctrlo:devfrom
droberts-ctrlo:autorecover-changes
Jan 26, 2026
Merged

Updated to create alert rather than modal for auto-recover [D: 831]#579
abeverley merged 3 commits intoctrlo:devfrom
droberts-ctrlo:autorecover-changes

Conversation

@droberts-ctrlo
Copy link
Contributor

No description provided.

@droberts-ctrlo droberts-ctrlo force-pushed the autorecover-changes branch 2 times, most recently from 6334d58 to 169fdc7 Compare January 12, 2026 15:17
- Add `Buildable` interface
- Add the `Hidable` abstract class for hiding elements
- Add the `ButtonBuilder` for building custom buttons
- Add the `AlertBuilder` abstract class for creating alerts, as well as a number of specific Alert types that can be built using this class
- Prevent alert showing when not on record pages
- Prevent modal showing when other alerts have values
- Reword values in alert and modal
- Update modal to show fields being changed
@droberts-ctrlo droberts-ctrlo force-pushed the autorecover-changes branch 2 times, most recently from 177fe9c to 345e3c9 Compare January 22, 2026 15:46
@abeverley abeverley merged commit 39ccefd into ctrlo:dev Jan 26, 2026
6 checks passed
@droberts-ctrlo droberts-ctrlo deleted the autorecover-changes branch February 9, 2026 09:10
droberts-ctrlo added a commit to droberts-ctrlo/GADS that referenced this pull request Feb 9, 2026
…trlo#579)

* Fix for Auto-Recover being invasive

- Add `Buildable` interface
- Add the `Hidable` abstract class for hiding elements
- Add the `ButtonBuilder` for building custom buttons
- Add the `AlertBuilder` abstract class for creating alerts, as well as a number of specific Alert types that can be built using this class
- Prevent alert showing when not on record pages
- Prevent modal showing when other alerts have values
- Reword values in alert and modal
- Update modal to show fields being changed

* Reword modal

* Updated cypress tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants