Skip to content

[PB-6328/PB-6333]: feat/validate file size before upload#1957

Draft
xabg2 wants to merge 2 commits into
masterfrom
feature/max-file-size-limit-per-tier
Draft

[PB-6328/PB-6333]: feat/validate file size before upload#1957
xabg2 wants to merge 2 commits into
masterfrom
feature/max-file-size-limit-per-tier

Conversation

@xabg2
Copy link
Copy Markdown
Contributor

@xabg2 xabg2 commented May 14, 2026

Description

Implemented a pre-validation step before starting uploads to filter out files that exceed the maximum upload size allowed for the user’s current tier.

If a file is larger than the allowed limit, a dialog is displayed informing the user about:

  • their current upload size limit
  • why the upload cannot proceed
  • the next available tier required to upload larger files

This prevents invalid uploads from starting and provides clearer upgrade guidance to the user.

Related Issues

Related Pull Requests

Checklist

  • Changes have been tested locally.
  • Unit tests have been written or updated as necessary.
  • The code adheres to the repository's coding standards.
  • Relevant documentation has been added or updated.
  • No new warnings or errors have been introduced.
  • SonarCloud issues have been reviewed and addressed.
  • QA Passed

Testing Process

Additional Notes

@xabg2 xabg2 self-assigned this May 14, 2026
@xabg2 xabg2 added the enhancement New feature or request label May 14, 2026
@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented May 14, 2026

Deploying drive-web with  Cloudflare Pages  Cloudflare Pages

Latest commit: 41cfb6c
Status: ✅  Deploy successful!
Preview URL: https://f8a838c0.drive-web.pages.dev
Branch Preview URL: https://feature-max-file-size-limit.drive-web.pages.dev

View logs

@sonarqubecloud
Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
60.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant