Skip to content

fix broken start state#87

Merged
naltatis merged 2 commits intomainfrom
fix/async_startup
Sep 15, 2025
Merged

fix broken start state#87
naltatis merged 2 commits intomainfrom
fix/async_startup

Conversation

@naltatis
Copy link
Copy Markdown
Member

The introduced initialRouteName leads to problems on startup because serverUrl is evaluated asynchronously (initially "unknown" > [real value]). Initial Route does not reflect the update. Introducing a dedicated loading state here to fix this.

\cc @Maschga

@naltatis naltatis added the bug Something isn't working label Sep 15, 2025
@naltatis
Copy link
Copy Markdown
Member Author

@Maschga musste das mit Initial Route wieder zurückbauen, da es mehrere Fälle (bspw. server löschen) gab wo wir dann in falschen Zuständen steckengeblieben sind. Das manuelle Navigieren zwischen Main/Server abhängig davon ob man den serverUrl Wert gerade neu setzt oder entfernt geht theoretisch. Erfordert aber viel Disziplin. ServerManual ist nun einfach in beiden Stacks enthalten. Damit klappt auch der Direkteinsprung.

Aus Nutzersicht wäre es im Fall "Direkteinsprung + serverUrl existiert" eigentlich sogar ganz gut, wenn erst noch ein Dialog kommt, der fragt ob man wirklich seine aktuelle Einstellung überschreiben möchte. Das könnte sich dann im "SettingsScreen" abspielen. Aber ist etwas für einen separaten PR.

@naltatis naltatis merged commit 15d04aa into main Sep 15, 2025
5 checks passed
@Maschga
Copy link
Copy Markdown
Collaborator

Maschga commented Sep 17, 2025

Der PR hat mutmaßlich auch #80 (comment) gelöst. 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants