Skip to content

Merge Nightly to Production#61

Merged
Adithyan-Dinesh-Trenser merged 15 commits intoproductionfrom
gradienthealth/studies-downloader-route
Feb 5, 2026
Merged

Merge Nightly to Production#61
Adithyan-Dinesh-Trenser merged 15 commits intoproductionfrom
gradienthealth/studies-downloader-route

Conversation

@Adithyan-Dinesh-Trenser
Copy link

@Adithyan-Dinesh-Trenser Adithyan-Dinesh-Trenser commented Dec 12, 2025

  • Support for updated review sheet template.

    • Updated the series filtering to work with the new review sheet.
  • Made the right form panel open by default when launching the viewer( in viewer and segmentation modes ).

  • OPFS management tool improvements

    • Added manual purge dropdown to delete OPFS files older than a specific time.
    • Displayed total file size under the table( bottom-center ).
    • Displayed the collective file size of selected rows next to the selected rows count( bottom-left ).
    • Fixed the issue with StudyDescription not included in the global search.
    • Fixed the issue with LastUpdated using the formatted value for sorting.
  • Added UserProject header to segmentation requests.

  • Bug fixes and optimisation.

  • Bug fixes and optimisation in cod-dicomweb-server package.

Associated PRs

Back merge the merge commits from production
* Optimized Series filtering in COD
* Modifications to support the additional sheet features and fixed the bugs
…being out of context in some sheet-integrated scenarios
…ndled the evaluate function for segmentation tools
* Added manual purge dropdown to delete OPFS files older than a specific time.
* Displayed total file size under the table(bottom-center).
* Displayed the collective file size of selected rows next to the selected rows count( bottom-left ).
* Fixed the issue with StudyDescription not included in the global search.
* Fixed the issue with LastUpdated using the formatted value for sorting.
@Adithyan-Dinesh-Trenser Adithyan-Dinesh-Trenser changed the title Support for more Google Sheet features Merge Nightly to Production Feb 3, 2026
entries.map(async subDirHandle => {
if (!maxAgeMs) {
await dirHandle.removeEntry(subDirHandle.name, { recursive: true });
}

Choose a reason for hiding this comment

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

maybe remove this if statement?

Choose a reason for hiding this comment

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

This if statement is necessary. I have added a comment to describe the use.

Copy link

@chengke2442 chengke2442 left a comment

Choose a reason for hiding this comment

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

LGTM. Let's merge this first to verify the functionality. We can monitor the performance in a real-world scenario and iterate if needed.

@Adithyan-Dinesh-Trenser Adithyan-Dinesh-Trenser merged commit 87da06f into production Feb 5, 2026
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.

2 participants