Skip to content

fix: Hide "Issues connecting?" on Splash Screen if no URL set - #1159

Open
Taarek wants to merge 11 commits into
fluxerapp:mainfrom
Taarek:fix-hide-status
Open

fix: Hide "Issues connecting?" on Splash Screen if no URL set#1159
Taarek wants to merge 11 commits into
fluxerapp:mainfrom
Taarek:fix-hide-status

Conversation

@Taarek

@Taarek Taarek commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Summary

  • What changed: Added a way to configure status page URL and incident history URL, so you can reuse the component showing up when the connection process takes longer than usual on the splash screen, otherwise it's hidden.
  • Why it is correct: This seemed to correct approach, instead of outright hiding it for all self-hosted instances
  • Risk: This does introduce a breaking change for the official instance, but @hampus-fluxer have been made aware of it.

Verification

  • Tests run: Biome and TS linting
  • Manual checks:
  • Screenshots or recordings:

Checklist

  • I understand every change in this PR.
  • I can explain what it does and why it is correct.
  • I disclosed any LLM coding help below.

LLM Disclosure

  • Had some help from Codex to make sure I did not miss some file to apply these changes to, instance configs resided in a lot more files than I initially thought when I started this PR.

closes #1259

@Taarek Taarek changed the title fix: Hide "Issues connecting"? on Splash Screen if no URL set fix: Hide "Issues connecting?" on Splash Screen if no URL set Jun 24, 2026
Taarek added 3 commits June 26, 2026 21:09
The app polls in the background for any active incidents or planned
maintenance, this is not needed for self hosted instances, so now it
checks this and doesn't run the background pollings if isSelfHosted is
`true`
BREAKING CHANGE: This does introduce a breaking change for the official
instance, as it will hide the "Issue connecting?" component if the new
config variable is not set
@Taarek
Taarek force-pushed the fix-hide-status branch from 789ebc2 to ed01019 Compare June 26, 2026 19:09

@Jiralite Jiralite left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • These need to be added to packages/config/src/config_loader/EnvironmentOverrides.ts
  • fluxer_app/src/features/user/state/StatusPage.ts polls hard-coded ExternalUrls.SERVICE_STATUS still. It will also fetch maintenance from that hard-coded value too. For correctness, let's swap them to the defined RuntimeConfig.statusPageUrl value. This is mainly a code correctness issue for the main instance where it will pointlessly poll if the main instance does not have one set
  • Please run the OpenAPI script generators. When schemas are updated, the JSON files need to be updated too (pnpm run openapi:generate)

@Taarek
Taarek requested a review from Jiralite July 1, 2026 11:07
@Taarek
Taarek force-pushed the fix-hide-status branch from 28cf24d to bfb4ca2 Compare July 1, 2026 11:43
@Taarek
Taarek force-pushed the fix-hide-status branch from 5272b56 to fff537e Compare July 1, 2026 12:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Self-Hosted] fluxerstatus.com hardcoded in loading screen

3 participants