Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

On deletion of integration credentials -> delete all associated webhooks first #8357

Closed
Tracked by #8358
Pwuts opened this issue Oct 16, 2024 — with Linear · 0 comments
Closed
Tracked by #8358

On deletion of integration credentials -> delete all associated webhooks first #8357

Pwuts opened this issue Oct 16, 2024 — with Linear · 0 comments
Assignees
@Pwuts
Labels
platform/backend AutoGPT Platform - Back end

Comments

@Pwuts Linear
Copy link
Member

Pwuts commented Oct 16, 2024 •
edited
Loading

On deletion of integration credentials, delete all associated webhooks first (both at the provider and in our DB), using those credentials.

The user has to be asked for confirmation before deleting in-use webhooks (which consequentially deactivates the linked agents).
@Pwuts Pwuts added the platform/backend AutoGPT Platform - Back end label Oct 16, 2024 — with Linear
This was referenced Oct 16, 2024
Closed
Closed
@Pwuts Pwuts mentioned this issue Nov 11, 2024
Merged
14 tasks
@Pwuts Pwuts self-assigned this Nov 21, 2024
Pwuts added a commit that referenced this issue Nov 21, 2024
@Pwuts
Remove webhooks before deleting credentials
8761c03 
Resolves #8357

- Add user flow to explicitly confirm deleting credentials linked to in-use webhooks
- Add logic to deregister, remove, and unlink webhooks before deleting parent credentials
- backend: Add `NeedConfirmation` exception
- frontend: Add `AlertDialog` UI component
@Pwuts Pwuts closed this as completed Nov 21, 2024
ntindle added a commit that referenced this issue Nov 27, 2024
@ntindle
feat(blocks): Various block QoL improvements (#8749)
fd6c1d9 
Co-authored-by: Reinier van der Leer <[email protected]>
Co-authored-by: Zamil Majdy <[email protected]>
Resolves #8357
Resolves #8738
aarushik93 pushed a commit that referenced this issue Dec 1, 2024
@ntindle @aarushik93
feat(blocks): Various block QoL improvements (#8749)
fc4d1f7 
Co-authored-by: Reinier van der Leer <[email protected]>
Co-authored-by: Zamil Majdy <[email protected]>
Resolves #8357
Resolves #8738
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Pwuts Pwuts

Labels
platform/backend AutoGPT Platform - Back end
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Pwuts