Skip to content

Merge with stage#135

Merged
susrisha merged 21 commits intomasterfrom
stage
Sep 26, 2025
Merged

Merge with stage#135
susrisha merged 21 commits intomasterfrom
stage

Conversation

@susrisha
Copy link
Copy Markdown
Contributor

This pull request introduces dataset viewer configuration support for project groups. The main changes include updating the database schema, API specification, and backend logic to allow storing and updating dataset viewer preferences (such as access flags and feedback turnaround time) per project group. The changes also add filtering capabilities for project groups based on dataset viewer access.

Database and API enhancements:

  • Added a new data_viewer_config JSON column to the project_group table to store dataset viewer preferences.
  • Updated database migration scripts for both up and down migrations to handle the new column. [1] [2]

API specification and DTO updates:

  • Extended the OpenAPI spec (user-management-spec.json) to include:
    • The new data_viewer_config property in relevant schemas.
    • The data_viewer_allowed query parameter for filtering.
    • The /api/v1/project-group/{projectGroupId}/dataset-viewer endpoint to update dataset viewer preferences.
    • The data_viewer schema describing allowed flags and feedback turnaround time. [1] [2] [3] [4] [5]
  • Added new DTOs for DatasetViewerDto and FeedbackTurnaroundTime with validation logic.
  • Updated existing DTOs and query params to include the new config and filtering options. [1] [2] [3] [4] [5]

Backend logic and service layer:

  • Implemented the updateDatasetViewerConfig method in the service layer to update the config for a project group.
  • Added a new controller endpoint to handle dataset viewer config updates, including request validation.
  • Updated service and controller interfaces to support the new functionality. [1] [2]

@susrisha susrisha requested a review from a team as a code owner September 24, 2025 03:31
@susrisha susrisha merged commit f769279 into master Sep 26, 2025
6 of 7 checks passed
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.

4 participants