Avoid blocking account config on status reads#24
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 01ef7340ff
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| state.configSource = "loading"; | ||
| render(); |
There was a problem hiding this comment.
Restore render after config load failures
When /api/config fails, returns non-OK, or the session expires and returns no accountOptions, this new loading state is rendered before the request but the failure/no-config paths only set state.configSource = "default" without calling render(). The UI therefore stays stuck on “Loading config” with switching disabled until the page is reloaded; re-render after the fallback state is applied.
Useful? React with 👍 / 👎.
Summary
Validation