Skip to content

Handle expired login while editing settings#364

Draft
Q1WP wants to merge 1 commit intobuildbotics:masterfrom
Q1WP:reconstruct/pr-settings-auth
Draft

Handle expired login while editing settings#364
Q1WP wants to merge 1 commit intobuildbotics:masterfrom
Q1WP:reconstruct/pr-settings-auth

Conversation

@Q1WP
Copy link
Copy Markdown
Contributor

@Q1WP Q1WP commented Mar 16, 2026

Test deployment containing this change: http://bbctrl.dyndns.org:8091/

Follow-up draft. Not intended for review until the macro PR is resolved.

This PR fixes Settings behavior when the user is logged out or the login
session expires before saving.

Changes:

  • make the Settings page read-only while unauthorized
  • disable Save while unauthorized
  • keep the leave-page dialog from acting like a failed save succeeded
  • prompt for login before retrying an expired-session save
  • show the API error message on save failure

Files:

  • src/js/view-settings.js
  • src/pug/templates/view-settings.pug
  • src/stylus/view-settings.styl

Validation:

  • node -c src/js/view-settings.js
  • manual test: lose auth, edit settings, confirm read-only behavior and
    login-before-save flow

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