Skip to content

[GRDM-57690] Update user Quota value when moving/copying files - #92

Draft
ndnhat1 wants to merge 6 commits into
developfrom
feature/nii_grdm_202603_step4/2.2_quota_calculation_when_move_file
Draft

ndnhat1 wants to merge 6 commits into
developfrom
feature/nii_grdm_202603_step4/2.2_quota_calculation_when_move_file

Conversation

@ndnhat1

@ndnhat1 ndnhat1 commented May 7, 2026 •

Copy link
Copy Markdown

Ticket

GRDM-57690

Purpose

  • Modify the quota calculation logic when moving/copying files from Extended Storage to Institutional Storage, as well as when moving/copying files between folders within the same storage.
  • Prevent moving or copying files that exceed the maximum storage limit file size.

Changes

  • Add max file size validation before performing move/copy operations.
  • Update _do_intra_move_or_copy logic to handle replaced_size during move/copy operations.

Side effects

QA Notes

Deployment Notes

Please ensure the following branches are deployed to their respective servers:

 + Raise an exception when dest_provider.metadata encounters an error
 + Handle next_token when moving or copying folders with more than 1000 files
 + Handle folder size limit and quota verification in tasks

@yacchin1205 yacchin1205 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

GRDM-57690にコメントを入れました。

@yacchin1205 yacchin1205 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

指摘事項を確認いただき、問題がなさそうならばApprove扱いでOKです。RDM-osf.io側と同様、対応するRDM-osf.ioへのリンク(これを必ずセットでデプロイすること)を、明記いただけると良いかと思います。

Comment thread waterbutler/server/api/v1/provider/movecopy.py Outdated

@yacchin1205 yacchin1205 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

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