Skip to content

Comments

Use hash-based routing to keep base URL visible throughout app#59

Closed
Copilot wants to merge 3 commits intofeature/multiuserfrom
copilot/improve-login-url-structure
Closed

Use hash-based routing to keep base URL visible throughout app#59
Copilot wants to merge 3 commits intofeature/multiuserfrom
copilot/improve-login-url-structure

Conversation

Copy link

Copilot AI commented Feb 19, 2026

Summary

Switches from BrowserRouter to HashRouter to keep base URL visible. Previously URLs showed http://localhost:9090/login and http://localhost:9090/app during navigation. Now routes appear after hash fragment: http://localhost:9090/#/login, http://localhost:9090/#/app.

Changes

  • InvokeAIUI.tsx: Replace BrowserRouter with HashRouter
  • LoginPage.tsx: Update redirect to window.location.href = '/#/app'
  • AdministratorSetup.tsx: Update redirect to window.location.href = '/#/login'

Uses window.location.href instead of window.location.hash to maintain full page reloads for multiuser isolation.

Related Issues / Discussions

None

QA Instructions

  1. Start app on feature/multiuser branch with multiuser enabled
  2. Navigate to login - URL should show http://localhost:9090/#/login
  3. Complete login - URL should show http://localhost:9090/#/app with page reload
  4. Test admin setup flow - redirects to /#/login with page reload
  5. Verify browser back/forward navigation works correctly
  6. Verify direct URL access (e.g., http://localhost:9090/#/login) works

Merge Plan

None

Checklist

  • The PR has a short but descriptive title, suitable for a changelog
  • Tests added / updated (if applicable)
  • ❗Changes to a redux slice have a corresponding migration
  • Documentation added / updated (if applicable)
  • Updated What's New copy (if doing a release after this PR)

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI and others added 2 commits February 19, 2026 12:55
Co-authored-by: lstein <111189+lstein@users.noreply.github.com>
…isolation

Co-authored-by: lstein <111189+lstein@users.noreply.github.com>
Copilot AI changed the title [WIP] Refactor login process to avoid user-visible URLs Use hash-based routing to keep base URL visible throughout app Feb 19, 2026
Copilot AI requested a review from lstein February 19, 2026 13:01
@lstein lstein closed this Feb 20, 2026
@lstein lstein deleted the copilot/improve-login-url-structure branch February 20, 2026 05:07
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