Skip to content

Conversation

@ChengShi-1
Copy link
Contributor

@ChengShi-1 ChengShi-1 commented Jul 25, 2025

What this PR does / why we need it:

In JSF, when we update Tags -> Edit Tags, there is a list of available categories populated from the dropdown lists, and some of them could be customized categories from other files in the same dataset.

Which issue(s) this PR closes:

Related Dataverse PRs:

Special notes for your reviewer:

RoleHelper for testing changed because Revert "feat: "Link Dataset/Dataverse" permission"

Suggestions on how to test this:

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

Additional documentation:

@ChengShi-1 ChengShi-1 linked an issue Jul 25, 2025 that may be closed by this pull request
@github-actions github-actions bot added FY26 Sprint 2 FY26 Sprint 2 (2025-07-16 - 2025-07-30) GREI Re-arch GREI re-architecture-related SPA.Q3.2025 SPA: File Page labels Jul 25, 2025
@ChengShi-1 ChengShi-1 added Size: 3 A percentage of a sprint. 2.1 hours. Original size: 3 labels Jul 25, 2025
@ChengShi-1 ChengShi-1 moved this to Ready for Review ⏩ in IQSS Dataverse Project Jul 25, 2025
@ChengShi-1 ChengShi-1 requested a review from Copilot July 25, 2025 14:45
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR introduces a new use case to retrieve available file categories for a dataset, which supports the JSF feature for populating category dropdown lists when editing file tags. The categories include both default categories and custom categories from other files in the same dataset.

  • Adds GetDatasetAvailableCategories use case and repository method
  • Updates documentation links for existing file metadata update use cases
  • Removes deprecated LinkDataverse and LinkDataset permissions from role definitions

Reviewed Changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
src/datasets/domain/useCases/GetDatasetAvailableCategories.ts Implements new use case to retrieve available file categories for a dataset
src/datasets/infra/repositories/DatasetsRepository.ts Adds repository method to fetch available categories from API endpoint
src/datasets/domain/repositories/IDatasetsRepository.ts Adds interface method for getting dataset available categories
src/datasets/index.ts Exports the new use case for public consumption
test/integration/datasets/DatasetsRepository.test.ts Adds integration tests for the new repository method
test/functional/datasets/GetDatasetAvailableCategories.test.ts Adds functional tests for the new use case
docs/useCases.md Documents the new use case with usage examples
src/files/domain/useCases/UpdateFileTabularTags.ts Fixes documentation URL fragment for tabular tags
src/files/domain/useCases/UpdateFileCategories.ts Fixes documentation URL fragment for file categories
test/testHelpers/roles/roleHelper.ts Removes deprecated link permissions from role definitions

@g-saracca g-saracca self-assigned this Jul 30, 2025
@g-saracca g-saracca moved this from Ready for Review ⏩ to In Review 🔎 in IQSS Dataverse Project Jul 30, 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.

Looks good, just one comment.

@cmbz cmbz added the FY26 Sprint 3 (2025-07-30 - 2025-08-13) label Jul 31, 2025
g-saracca
g-saracca previously approved these changes Jul 31, 2025
@ofahimIQSS
Copy link
Contributor

hello! can you resolve the conflicts please :)

@ofahimIQSS ofahimIQSS self-assigned this Aug 13, 2025
@ofahimIQSS ofahimIQSS moved this from Ready for QA ⏩ to QA ✅ in IQSS Dataverse Project Aug 13, 2025
@cmbz cmbz added the FY26 Sprint 4 FY26 Sprint 4 (2025-08-13 - 2025-08-27) label Aug 14, 2025
@ofahimIQSS ofahimIQSS assigned ChengShi-1 and unassigned ofahimIQSS Aug 15, 2025
@ofahimIQSS ofahimIQSS moved this from QA ✅ to Ready for Review ⏩ in IQSS Dataverse Project Aug 15, 2025
@github-project-automation github-project-automation bot moved this from Ready for Review ⏩ to Ready for QA ⏩ in IQSS Dataverse Project Aug 15, 2025
@g-saracca
Copy link
Contributor

hello! can you resolve the conflicts please :)

All yours @ofahimIQSS

@ofahimIQSS
Copy link
Contributor

looks good - merging

@ofahimIQSS ofahimIQSS merged commit 3219a71 into develop Aug 15, 2025
5 checks passed
@ofahimIQSS ofahimIQSS deleted the 336-file-use-cases-get-available-dataset-file-categories branch August 15, 2025 20:59
@github-project-automation github-project-automation bot moved this from Ready for QA ⏩ to Merged 🚀 in IQSS Dataverse Project Aug 15, 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

Projects

Status: Done 🧹

Development

Successfully merging this pull request may close these issues.

File: Use Cases Get Available Dataset File Categories

5 participants