Skip to content

fix: enable Codex Apply/Reset buttons when CLI is installed (closes #591)#606

Open
anuragg-saxenaa wants to merge 1 commit intodecolua:masterfrom
anuragg-saxenaa:fix/issue-591-codex-cli-apply-reset
Open

fix: enable Codex Apply/Reset buttons when CLI is installed (closes #591)#606
anuragg-saxenaa wants to merge 1 commit intodecolua:masterfrom
anuragg-saxenaa:fix/issue-591-codex-cli-apply-reset

Conversation

@anuragg-saxenaa
Copy link
Copy Markdown
Contributor

Closes #591

Issue: Codex CLI card showed Connected but Apply and Reset buttons were unavailable/disabled, making it impossible to configure or reset settings even when the CLI was installed.

Root Cause:

  • Apply button was disabled when selectedApiKey was empty — but sk_9router is a valid default key for local 9router, so it should be enabled even without an explicit selection
  • Reset button was disabled when has9Router was false — but users should always be able to reset, even if 9router was not previously configured

Fix:

  • Apply: enable when model is selected AND (API key is selected OR no cloud enabled with sk_9router default OR no keys available)
  • Reset: always enabled when CLI is installed (remove the has9Router check)
  • Buttons remain disabled while loading (applying/restoring states)

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.

Codex CLI card shows "Connected" but Apply/Reset are unavailable in Dashboard

1 participant