Skip to content

Conversation

@adzhindzhi
Copy link
Contributor

@adzhindzhi adzhindzhi commented Oct 23, 2025

Resolves:

UEPR-380

Changes:

There are currently two approaches in discussion: Display blocking modals vs. Display blocking pages

  • The first commit completes the modal approach implementation, showing a blocking modal when opening the editor as well (the remaining was already implemented with the non-blocking ToU modals). This solution has a couple of drawbacks - it’s easier to bypass, can conflict with other modals, and currently lacks a logout option.
  • The second commit reverts the modal implementation in favor of blocking pages. While this approach has some UX downsides, it is a bit more secure, harder to bypass, and supports logging out without additional work.

Test Coverage:

  • Current test cases complete successfully

Page vs. Modal approach UI comparison:

image image

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.

1 participant