Skip to content

fix(ui): update delete API key confirmation modal - #1106

Open
rachittaneja56-max wants to merge 1 commit into
stagefrom
feat/issue-1099-update-delete-api-key-modal
Open

rachittaneja56-max wants to merge 1 commit into
stagefrom
feat/issue-1099-update-delete-api-key-modal

Conversation

@rachittaneja56-max

Copy link
Copy Markdown

Description

Closes #1099

Updated the delete API key confirmation modal to match the approved design.

The new layout is opt-in and enabled only for API-key deletion. Existing account deletion and admin milestone modals keep their current layout and behavior.

Preserved:

  • API-key name confirmation
  • Disabled confirm button
  • Loading state
  • Cancel behavior
  • Delete request

What type of PR is this? (Check all applicable)

  • 🍕 Feature
  • 🐛 Bug Fix
  • 📄 Documentation Update
  • 👨‍💻 Code Refactor
  • 🔥 Performance Improvements
  • ✅ Test
  • 🛠️ CI/CD

Screenshots (if applicable)

Screen.Recording.2026-09-01.013152.mp4

Checklist

  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing focused unit tests pass locally with my changes

Note: The repository-wide Prettier check reports 15 unrelated formatting issues outside this PR.

@vercel

vercel Bot commented Aug 31, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
openlogo-prod-web Ready Ready Preview Aug 31, 2026 8:34pm
openlogo-staging-web Ready Ready Preview Aug 31, 2026 8:34pm

@github-actions github-actions Bot added the enhancement New feature or request label Aug 31, 2026
@sonarqubecloud

Copy link
Copy Markdown

@AditSingh7 AditSingh7 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks Good

@0-mstrmind 0-mstrmind left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All good :-)

This branch was successfully deployed

2 active deployments
Preview – openlogo-staging-web — 961a3181 Deployed Aug 31, 2026 by vercel[bot]
Preview – openlogo-prod-web — 961a3181 Deployed Aug 31, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update delete api key modal

3 participants