Skip to content

Conversation

@frenck
Copy link
Member

@frenck frenck commented Nov 20, 2025

Proposed change

When an integration requires reauthentication, a reauth card appears at the top of the integrations dashboard. Previously, users could only delete the config entry from the original integration card lower in the list, which was confusing and hard to find.

This PR adds a delete option directly to the reauth card's menu, allowing users to remove the config entry from the prominent card that's "in their face" prompting for action.

CleanShot 2025-11-20 at 21 54 12@2x

Type of change

  • Dependency upgrade
  • Bugfix (non-breaking change which fixes an issue)
  • New feature (thank you!)
  • Breaking change (fix/feature causing existing functionality to break)
  • Code quality improvements to existing code or addition of tests

Example configuration

N/A - This is a UI enhancement

Additional information

  • This PR fixes or closes issue: fixes #
  • This PR is related to issue or discussion:
  • Link to documentation pull request: N/A

Implementation details:

  • Delete menu item appears in the three-dot menu on reauth cards only
  • Uses the same confirmation dialogs and localization as integration entry delete
  • Handles application credentials cleanup
  • Shows restart notifications when required
  • Styled in red (warning color) to indicate destructive action
  • Gets the entry_id from flow.context.entry_id which is always present for reauth flows

Checklist

  • The code change is tested and works locally.
  • There is no commented out code in this PR.
  • Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

Users can now delete config entries directly from the reauth card that appears at the top of the integrations dashboard, instead of having to scroll down to find the original integration card.

The delete option:
- Appears in the three-dot menu on reauth cards
- Shows a confirmation dialog before deletion
- Handles application credentials cleanup
- Shows restart notifications when required
- Uses the same styling and localization as the integration entry delete

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
MindFreeze
MindFreeze previously approved these changes Nov 21, 2025
@MindFreeze MindFreeze merged commit c679e31 into dev Nov 21, 2025
15 checks passed
@MindFreeze MindFreeze deleted the frenck-2025-0550 branch November 21, 2025 11:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants