Skip to content

fix: reload dashboard under http node root#2152

Open
Saithej2k wants to merge 1 commit into
FlowFuse:mainfrom
Saithej2k:fix/reload-with-http-node-root
Open

fix: reload dashboard under http node root#2152
Saithej2k wants to merge 1 commit into
FlowFuse:mainfrom
Saithej2k:fix/reload-with-http-node-root

Conversation

@Saithej2k

Copy link
Copy Markdown

Summary

  • Build forced reload URLs from the dashboard base path instead of always using /dashboard.
  • Use the _setup response URL when unauthenticated responses arrive before setup is stored.

Testing

  • git diff --check
  • node --check ui/src/main.mjs

Closes #2149

@Saithej2k Saithej2k marked this pull request as ready for review June 27, 2026 23:05
@colinl

colinl commented Jun 28, 2026

Copy link
Copy Markdown
Collaborator

Is the change to use the setup url in the case of authentication error anything to do with issue #2149 or is that to address a different issue?

@Saithej2k Saithej2k force-pushed the fix/reload-with-http-node-root branch from 9c2bc83 to 0dcf66c Compare June 29, 2026 18:15
@Saithej2k

Copy link
Copy Markdown
Author

Good point. I narrowed that part out so the auth path no longer depends on the _setup response URL. The reload URL now comes from the stored setup base path when available, or from the current dashboard path, which keeps the change focused on #2149.

@colinl

colinl commented Jul 2, 2026

Copy link
Copy Markdown
Collaborator

While you have your brain in gear for the dashboard url, you might like to have a look at #2132.

Does the Open Dashboard button in the editor work if the root is altered?

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.

forcePageReload ignores httpNodeRoot

2 participants