Skip to content

Add total file size to export selected data summary#4739

Merged
NoopDog merged 2 commits intomainfrom
fran/4738-file-size
Mar 30, 2026
Merged

Add total file size to export selected data summary#4739
NoopDog merged 2 commits intomainfrom
fran/4738-file-size

Conversation

@frano-m
Copy link
Copy Markdown
Contributor

@frano-m frano-m commented Mar 30, 2026

Summary

  • Add total file size to the selected data summary on the AnVIL export page
  • Binds totalFileSize from the summary API response and formats it using formatFileSize

Closes #4738

Test plan

  • Verify file size appears in the selected data summary on the cohort export page
  • Verify file size is formatted in human-readable units (e.g., GB, TB)

🤖 Generated with Claude Code

image image

Copy link
Copy Markdown
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

Adds the total file size returned by the AnVIL-CMG summary API to the “Export Selected Data” summary so users can see the total data volume being exported.

Changes:

  • Bind totalFileSize from the summary API response into the export summary view model.
  • Add a new summary key/label for file size and include it in the export summary map.
  • Format the total size using formatFileSize.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
app/viewModelBuilders/azul/anvil-cmg/common/summaryMapper/summaryMapper.ts Binds totalFileSize and adds a formatted “File Size” entry to the export summary.
app/viewModelBuilders/azul/anvil-cmg/common/summaryMapper/entities.ts Extends FileSummary and SUMMARY to include the new file size summary field/key.
app/viewModelBuilders/azul/anvil-cmg/common/summaryMapper/constants.ts Adds default value and display label for the new “File Size” summary row.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@NoopDog NoopDog merged commit 1fab2f9 into main Mar 30, 2026
3 checks passed
@frano-m frano-m deleted the fran/4738-file-size branch March 30, 2026 08:08
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.

[AnVIL DX] Add total file size to export selected data summary

3 participants