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.

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

feat: Validate workspace relationship when deleting a vcs provider #1675

Merged
merged 3 commits into from
Dec 30, 2024

Conversation

alfespa17
Copy link
Member

@alfespa17 alfespa17 commented Dec 30, 2024

This will show a message that a VCS provider can't be deleted because there a workspaces that are using it

image

This will only work for workspace that were deleted after this was implemented #1502 because there could be some deleted workspaces that could have the reference in the database, for previous workspaces it will be required to connect to the database and remove the reference manually

Fix #1666

@alfespa17 alfespa17 merged commit 4e014b9 into main Dec 30, 2024
5 checks passed
@alfespa17 alfespa17 deleted the feat/message-vcs branch December 30, 2024 15:44
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.

Removing VCS from GUI with connected workspaces doesn't result in error message GUI
1 participant