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

Resolve dataset share checks when deleting dataset #554

Merged
merged 1 commit into from
Jul 10, 2023

Conversation

noah-paige
Copy link
Contributor

Feature or Bugfix

  • Bugfix

Detail

  • Fix query to check dataset shares before deleting a dataset

The existingSharedItems property of ShareObject was not resolving correctly to a True or False value but rather to None. The impact is:

  1. When checking a dataset's shares before deleting the dataset - we would not raise an exception for datasets that did in fact have existing shared items.

  2. Additionally, when deleting a dataset with associated shared objects created - we would not successfully delete the associated share object and share object items and not clean up rows in the DB appropriately. This caused issues when viewing the Share Tab as the dataset does not exist but is still referenced in the outdated ShareObject and throws an error.

Relates

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Copy link
Contributor

@dlpzx dlpzx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, thank you!

@noah-paige noah-paige merged commit 558e2bc into main Jul 10, 2023
@dlpzx dlpzx deleted the 544-delete-dataset-resolve-delete-shares branch July 20, 2023 09:17
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.

User unable to submit a new share request when an earlier requested dataset was deleted
2 participants