Hide current status rows in public preview#29
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 81ac62e746
ℹ️ 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".
| const currentEntry = currentEntryForAccount(state.selected, account); | ||
| const currentMode = normalizeExecutionMode(currentEntry?.execution_mode, currentEntry?.dry_run_only); | ||
| el("mode-pill").textContent = currentMode ? modeLabel(currentMode) : t("notRead"); | ||
| el("mode-pill").textContent = modeLabel(inputs.execution_mode); |
There was a problem hiding this comment.
Keep the current mode in the signed-in summary
When a signed-in user changes the mode selector before dispatching, this now renders the pill from buildInputs() (the pending selection) while the panel title remains Current Config and the rows still include the current strategy. For an account whose stored mode is paper, clicking Live immediately makes the current-config pill say Live even though /api/config still reports paper, so the UI misstates the active mode until the workflow actually completes.
Useful? React with 👍 / 👎.
Summary
Validation