Thanks to visit codestin.com
Credit goes to github.com

Skip to content

feat: delete AI agent slack integrations when removing Slack installation #14980

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
Merged
merged 1 commit into from
May 23, 2025
Merged

feat: delete AI agent slack integrations when removing Slack installation #14980

merged 1 commit into from
May 23, 2025

Conversation

@notgiorgi
Copy link
Contributor

Closes: #14934

Description:

Added functionality to delete AI agent Slack integrations when a Slack installation is removed from an organization. This ensures that when a user disconnects Slack from their organization, all related AI agent Slack integrations are properly cleaned up as well.

@notgiorgi notgiorgi mentioned this pull request May 23, 2025
Merged
@notgiorgi Graphite App
Copy link
Contributor Author

notgiorgi commented May 23, 2025 •
edited
Loading

This stack of pull requests is managed by Graphite. Learn more about stacking.

@notgiorgi notgiorgi requested review from IrakliJani and joaoviana May 23, 2025 12:02
joaoviana
joaoviana approved these changes May 23, 2025
View reviewed changes
@notgiorgi notgiorgi removed the request for review from IrakliJani May 23, 2025 12:55
@notgiorgi notgiorgi changed the base branch from 05-22-fix_agents_should_be_shown_even_if_there_s_no_slack_integration to graphite-base/14980 May 23, 2025 13:43
@notgiorgi notgiorgi force-pushed the graphite-base/14980 branch from 5711853 to d574fe2 Compare May 23, 2025 13:43
@notgiorgi notgiorgi force-pushed the feat_delete_AI_agent_slack_integrations_when_removing_Slack_installation branch from b42adfd to aa4029b Compare May 23, 2025 13:43
@graphite-app graphite-app bot changed the base branch from graphite-base/14980 to main May 23, 2025 13:44
@notgiorgi notgiorgi force-pushed the feat_delete_AI_agent_slack_integrations_when_removing_Slack_installation branch from aa4029b to 924fae9 Compare May 23, 2025 13:44
@owlas owlas deployed to feat_delete_AI_agent_slack_integrations_when_removing_Slack_installation - jaffle_db_pg_13 PR #14980 May 23, 2025 13:44 — with Render Active
@owlas owlas temporarily deployed to feat_delete_AI_agent_slack_integrations_when_removing_Slack_installation - headless-browser PR #14980 May 23, 2025 13:44 — with Render Destroyed
@owlas owlas temporarily deployed to feat_delete_AI_agent_slack_integrations_when_removing_Slack_installation - lightdash PR #14980 May 23, 2025 13:52 — with Render Destroyed
@notgiorgi Graphite App
Copy link
Contributor Author

notgiorgi commented May 23, 2025 •
edited
Loading

Merge activity

  • May 23, 2:46 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • May 23, 2:46 PM UTC: @notgiorgi merged this pull request with Graphite.

@notgiorgi notgiorgi merged commit 1591e45 into main May 23, 2025
60 of 65 checks passed
@notgiorgi notgiorgi deleted the feat_delete_AI_agent_slack_integrations_when_removing_Slack_installation branch May 23, 2025 14:46
lightdash-bot pushed a commit that referenced this pull request May 23, 2025
@semantic-release-bot
chore(release): 0.1647.0
0347fe8 
# [0.1647.0](0.1646.0...0.1647.0) (2025-05-23)

### Features

* delete AI agent slack integrations when removing Slack installation ([#14980](#14980)) ([1591e45](1591e45)), closes [#14934](#14934)
@lightdash-bot
Copy link
Collaborator

🎉 This PR is included in version 0.1647.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@joaoviana joaoviana joaoviana approved these changes

Assignees

@notgiorgi notgiorgi

Labels

released

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Decouple Slack being required to see List of AI Agents

5 participants

@notgiorgi @lightdash-bot @joaoviana @owlas