Skip to content

feat(dashboard): move Danger Zone below API Keys in project General settings#934

Merged
zbigniewsobiecki merged 1 commit intodevfrom
feature/reorder-danger-zone-below-api-keys
Mar 17, 2026
Merged

feat(dashboard): move Danger Zone below API Keys in project General settings#934
zbigniewsobiecki merged 1 commit intodevfrom
feature/reorder-danger-zone-below-api-keys

Conversation

@aaight
Copy link
Copy Markdown
Collaborator

@aaight aaight commented Mar 17, 2026

Summary

  • Reorders the "Danger Zone" card to appear below the "API Keys" card in the ProjectGeneralForm component
  • The <AlertDialog> (delete confirmation dialog) moves with the Danger Zone card
  • No logic, state, or API changes — purely a JSX layout reorder

New section order in General tab:

  1. Project Identity (form)
  2. Budget & Limits (form)
  3. Progress Monitoring (form)
  4. Save/Reset buttons
  5. API Keys card
  6. Danger Zone card (with red border) ← moved to bottom
  7. Delete confirmation AlertDialog

Test plan

  • Navigate to dashboard → select a project → General tab
  • Verify "API Keys" card appears directly after the Save/Reset buttons
  • Verify "Danger Zone" card (red border) appears at the very bottom
  • Verify "Delete Project" button opens confirmation dialog and functions correctly

Trello card: https://trello.com/c/hiAjQWY8/452-danger-zone-should-be-under-the-api-key-configuration-in-general-in-dashboard-in-particular-project

🤖 Generated with Claude Code

🕵️ claude-code · claude-sonnet-4-6 · run details

Copy link
Copy Markdown
Collaborator

@nhopeatall nhopeatall left a comment

Choose a reason for hiding this comment

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

LGTM — clean JSX block reorder. The API Keys card and Danger Zone card swap is a character-identical move with no logic, state, or API changes. Final layout order matches the PR description. All CI checks pass.

🕵️ claude-code · claude-opus-4-6 · run details

@zbigniewsobiecki zbigniewsobiecki merged commit 7210a99 into dev Mar 17, 2026
5 checks passed
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.

3 participants