Skip to content

Conversation

@ChengShi-1
Copy link
Contributor

@ChengShi-1 ChengShi-1 commented Aug 1, 2025

What this PR does / why we need it:

Get dataset categories for users, in Dataset Page and File Page, once they open the Edit Tags Modal
PACKAGE UPDATE NEEDED, WAITING IQSS/dataverse-client-javascript#344 THIS TO BE MERGED

Which issue(s) this PR closes:

Special notes for your reviewer:

Besides this function, also did some other changes

Suggestions on how to test this:

test the modal for both dataset page, and file page

image image image

check if the categories is automatically populated in Edit File -> Tags modal. Upload two files, type some custom file tags to file 1, check if file 2 has the File Tags dropdown options of these custom file tags.

Does this PR introduce a user interface change? If mockups are available, please link/include them here:

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

Additional documentation:

@github-actions github-actions bot added 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.5 File Page: Edit File Tags SPA: File Page labels Aug 1, 2025
@coveralls
Copy link

coveralls commented Aug 1, 2025

Coverage Status

coverage: 97.474% (+0.1%) from 97.368%
when pulling 66bba01 on 772-file-edit-tags-populated-the-categories-dropdown-with-results-from-api
into 6d08b5e on develop.

@ChengShi-1 ChengShi-1 added Size: 3 A percentage of a sprint. 2.1 hours. Original size: 3 labels Aug 1, 2025
@ChengShi-1 ChengShi-1 moved this to In Progress 💻 in IQSS Dataverse Project Aug 1, 2025
@ChengShi-1 ChengShi-1 self-assigned this Aug 4, 2025
@ChengShi-1 ChengShi-1 moved this from In Progress 💻 to Ready for Review ⏩ in IQSS Dataverse Project Aug 4, 2025
@ChengShi-1 ChengShi-1 removed their assignment Aug 4, 2025
@ChengShi-1 ChengShi-1 marked this pull request as ready for review August 4, 2025 17:34
@g-saracca g-saracca self-assigned this Aug 6, 2025
@g-saracca g-saracca moved this from Ready for Review ⏩ to In Review 🔎 in IQSS Dataverse Project Aug 6, 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.

Looking really good, just some comments.

@ChengShi-1 ChengShi-1 removed their assignment Aug 7, 2025
@ChengShi-1 ChengShi-1 requested a review from g-saracca August 7, 2025 15:59
g-saracca
g-saracca previously approved these changes Aug 7, 2025
@github-project-automation github-project-automation bot moved this from In Review 🔎 to Ready for QA ⏩ in IQSS Dataverse Project Aug 7, 2025
@g-saracca g-saracca removed their assignment Aug 7, 2025
@cmbz cmbz added the FY26 Sprint 4 FY26 Sprint 4 (2025-08-13 - 2025-08-27) label Aug 14, 2025
@ofahimIQSS
Copy link
Contributor

Hello! PR has branch conflicts- please resolve. Thanks!

@ofahimIQSS ofahimIQSS moved this from Ready for QA ⏩ to Ready for Review ⏩ in IQSS Dataverse Project Aug 15, 2025
@g-saracca
Copy link
Contributor

I will review again after merge conflicts are solved 👍🏼

@github-project-automation github-project-automation bot moved this from Ready for Review ⏩ to Ready for QA ⏩ in IQSS Dataverse Project Aug 15, 2025
@ChengShi-1 ChengShi-1 removed their assignment Aug 15, 2025
@ofahimIQSS ofahimIQSS self-assigned this Aug 18, 2025
@ofahimIQSS ofahimIQSS moved this from Ready for QA ⏩ to QA ✅ in IQSS Dataverse Project Aug 18, 2025
@ofahimIQSS
Copy link
Contributor

Looks good to me - merging

@ofahimIQSS ofahimIQSS merged commit 285b84a into develop Aug 18, 2025
14 checks passed
@ofahimIQSS ofahimIQSS deleted the 772-file-edit-tags-populated-the-categories-dropdown-with-results-from-api branch August 18, 2025 13:23
@github-project-automation github-project-automation bot moved this from QA ✅ to Merged 🚀 in IQSS Dataverse Project Aug 18, 2025
@ofahimIQSS ofahimIQSS removed their assignment Aug 18, 2025
@scolapasta scolapasta moved this from Merged 🚀 to Done 🧹 in IQSS Dataverse Project Aug 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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: File Page SPA.Q3.2025.5 File Page: Edit File Tags

Projects

Status: Done 🧹

Development

Successfully merging this pull request may close these issues.

File - Edit Tags: populate the categories dropdown with results from api

6 participants