Skip to content

fix(GAT-9715): Allow deleting a team whose only datasets are archived - #1889

Merged
equinoxmatt merged 1 commit into
fix/GAT-9715_audit_successful_team_deletefrom
fix/GAT-9715_allow_team_delete_with_archived_datasets
Oct 22, 2026
Merged

equinoxmatt merged 1 commit into
fix/GAT-9715_audit_successful_team_deletefrom
fix/GAT-9715_allow_team_delete_with_archived_datasets

Conversation

@equinoxmatt

Copy link
Copy Markdown
Contributor

Describe your changes

Archived datasets no longer block team deletion. Active and draft datasets still do. Archived datasets keep their team_id and remain linked to the soft-deleted team.

Issue ticket link

https://hdruk.atlassian.net/browse/GAT-9715

Environment / Configuration changes (if applicable)

Requires migrations being run?

If not using the pre-push hook. Confirm tests pass:

Checklist before requesting a review

  • I have performed a self-review of my code
  • I have added appropriate unit tests
  • I have created mocks for unit tests (where appropriate)
  • I have added appropriate Behat tests to confirm AC (if applicable)
  • I have added Swagger annotations for new endpoints (if applicable)
  • I have added audit logs for new operation logic (if applicable)
  • I have added new environment variables to the .env.example file (if applicable)
  • I have added new environment variables to terraform repository (if applicable)

Archived datasets no longer block team deletion. Active and draft
datasets still do. Archived datasets keep their team_id and remain
linked to the soft-deleted team.
@gh-actions-pipelines-app

Copy link
Copy Markdown

🎉 Great job! Your PR title follows the correct format. 🚀

@equinoxmatt
equinoxmatt merged commit 2b0fa1d into dev Oct 22, 2026
3 checks passed
@equinoxmatt
equinoxmatt deleted the fix/GAT-9715_allow_team_delete_with_archived_datasets branch October 22, 2026 13:43

This branch was successfully deployed

1 active deployment
dev — 6d743232 Deployed Oct 22, 2026 by equinoxmatt via test #6984
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.

2 participants