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

Extracted samples projects issue, projects_samples table needs syncing #1136

Closed
stuzart opened this issue Jul 29, 2022 · 0 comments · Fixed by #1608
Closed

Extracted samples projects issue, projects_samples table needs syncing #1136

stuzart opened this issue Jul 29, 2022 · 0 comments · Fixed by #1608
Assignees
Labels
Milestone

Comments

@stuzart
Copy link
Member

stuzart commented Jul 29, 2022

Extracted samples have their projects and permissions delegated and bound to the originating data file.
Unfortunately, this means that for those samples, the projects_samples table is blank leading to a problem with Project.samples and Sample filtering by project.

A simple solution is to copy across the projects into the table when the sample is created, and also whenever the data file projects are changed.

@stuzart stuzart added this to the v1.13.0 milestone Jul 29, 2022
@stuzart stuzart modified the milestones: v1.13.0, v1.13.1 Sep 26, 2022
@stuzart stuzart modified the milestones: v1.13.1, 1.13.2 Feb 3, 2023
@stuzart stuzart added the samples label Feb 8, 2023
@stuzart stuzart modified the milestones: v1.13.2, v1.14.0 Feb 8, 2023
@stuzart stuzart modified the milestones: v1.14.0, v1.15.0 Jul 5, 2023
@floradanna floradanna moved this to Requirements in DataHub Jul 26, 2023
@floradanna floradanna moved this from Requirements to Backlog in DataHub Aug 1, 2023
@fherreazcue fherreazcue self-assigned this Oct 4, 2023
@fherreazcue fherreazcue linked a pull request Oct 6, 2023 that will close this issue
@floradanna floradanna moved this from Backlog to Merged in DataHub Nov 8, 2023
@kdp-cloud kdp-cloud moved this from Merged to Released in DataHub Jul 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Released
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants