Skip to content

Conversation

@ekraffmiller
Copy link
Contributor

@ekraffmiller ekraffmiller commented Aug 5, 2025

What this PR does / why we need it:

SPA Q3 Feature - Add Link and Unlink Collection features

Which issue(s) this PR closes:

Related Dataverse PRs:

Special notes for your reviewer:

This PR uses an API in the above pull request, need to wait for it to be merged first. Also, after the Dataverse PR is merged, need to change the Dataverse testing image back to unstable.

Suggestions on how to test this:

Review the test code, or test by creating your own linked collections and datasets in the UI, then run the API against that.

Is there a release notes update needed for this change?:

no

Additional documentation:

@github-actions github-actions bot added FY26 Sprint 1 FY26 Sprint 1 (2025-07-02 - 2025-07-16) FY26 Sprint 2 FY26 Sprint 2 (2025-07-16 - 2025-07-30) FY26 Sprint 3 (2025-07-30 - 2025-08-13) GREI Re-arch GREI re-architecture-related SPA.Q3.2025.2 Link Collection labels Aug 5, 2025
@ekraffmiller ekraffmiller moved this to Ready for Review ⏩ in IQSS Dataverse Project Aug 12, 2025
@ekraffmiller ekraffmiller added Size: 3 A percentage of a sprint. 2.1 hours. Original size: 3 labels Aug 12, 2025
@ekraffmiller ekraffmiller marked this pull request as ready for review August 12, 2025 18:23
@cmbz cmbz added the FY26 Sprint 4 FY26 Sprint 4 (2025-08-13 - 2025-08-27) label Aug 14, 2025
@ekraffmiller ekraffmiller changed the title Add linkCollection and unlinkCollection use cases Add linkCollection, unlinkCollection and getCollectionLinks use cases Aug 14, 2025
@g-saracca g-saracca self-assigned this Aug 14, 2025
@g-saracca g-saracca moved this from Ready for Review ⏩ to In Review 🔎 in IQSS Dataverse Project Aug 14, 2025
Copy link
Contributor

@g-saracca g-saracca left a comment

Choose a reason for hiding this comment

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

Overall looks good, some comments and we need documentation about new use cases.
And we have some merge conflicts.

@ekraffmiller
Copy link
Contributor Author

@g-saracca thanks for your review! I think I addressed everything.

@ekraffmiller ekraffmiller removed their assignment Aug 18, 2025
Copy link
Contributor

@g-saracca g-saracca left a comment

Choose a reason for hiding this comment

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

Awesome! Approving!

@github-project-automation github-project-automation bot moved this from In Review 🔎 to Ready for QA ⏩ in IQSS Dataverse Project Aug 18, 2025
@g-saracca g-saracca removed their assignment Aug 18, 2025
@ofahimIQSS
Copy link
Contributor

Test passing - merging

@ofahimIQSS ofahimIQSS merged commit 204796f into develop Aug 18, 2025
5 checks passed
@github-project-automation github-project-automation bot moved this from Ready for QA ⏩ to Merged 🚀 in IQSS Dataverse Project Aug 18, 2025
@scolapasta scolapasta moved this from Merged 🚀 to Done 🧹 in IQSS Dataverse Project Aug 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

FY26 Sprint 1 FY26 Sprint 1 (2025-07-02 - 2025-07-16) FY26 Sprint 2 FY26 Sprint 2 (2025-07-16 - 2025-07-30) FY26 Sprint 3 (2025-07-30 - 2025-08-13) FY26 Sprint 4 FY26 Sprint 4 (2025-08-13 - 2025-08-27) GREI Re-arch GREI re-architecture-related Original size: 3 Size: 3 A percentage of a sprint. 2.1 hours. SPA.Q3.2025.2 Link Collection

Projects

Status: Done 🧹

Development

Successfully merging this pull request may close these issues.

Create Link Collection and Unlink Collection Use Cases

5 participants