Skip to content

web: indicate that it is in read-only mode #1434

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

Open
sudoforge opened this issue May 17, 2025 · 2 comments
Open

web: indicate that it is in read-only mode #1434

sudoforge opened this issue May 17, 2025 · 2 comments
Labels
area/ui/web Relates to the Web UI kind/feature Relates to a new feature priority/backlog Higher priority than priority/awaiting-more-evidence

Comments

@sudoforge
Copy link
Contributor

right now, there is no visual indication (e.g. a notification in the corner) indicating that the web ui is in read-only mode. we should add support for this.

@sudoforge sudoforge added area/ui/web Relates to the Web UI kind/feature Relates to a new feature priority/backlog Higher priority than priority/awaiting-more-evidence labels May 17, 2025
@MichaelMure
Copy link
Contributor

Eventually that would be like in most apps: showing a login button... but that's when we have external auth

@sudoforge
Copy link
Contributor Author

yeah, i see in the code that the idea behind the current impl of --read-only mode implies that it's what would be used to support external auth, but:

  1. we don't have that in place today, but more critically
  2. i think keeping a read-only mode around is actually very useful, to support something like web: run a public deployment of the latest release in --read-only mode #1432 where a consumer wants to expose it publicly, but not allow any interaction at all. so perhaps when we do implement external auth, we keep this around anyway.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ui/web Relates to the Web UI kind/feature Relates to a new feature priority/backlog Higher priority than priority/awaiting-more-evidence
Projects
None yet
Development

No branches or pull requests

2 participants