Skip to content

[PB-5957]: fix/remove max space occupied pre-check for shared uploads - #1931

Merged
CandelR merged 5 commits into
masterfrom
fix/remove-is-upload-allowed-for-shared-uploads
Apr 27, 2026
Merged

[PB-5957]: fix/remove max space occupied pre-check for shared uploads#1931
CandelR merged 5 commits into
masterfrom
fix/remove-is-upload-allowed-for-shared-uploads

Conversation

@xabg2

@xabg2 xabg2 commented Apr 14, 2026

Copy link
Copy Markdown
Contributor

Description

Removing the pre-check of max space occupied when the user is uploading items on shared folders. This was causing problems when a user with EDITOR role is uploading items, because we were checking if the user has free space in his account, not the owner account.

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 Apr 14, 2026
@xabg2 xabg2 added the bug Something isn't working label Apr 14, 2026
@cloudflare-workers-and-pages

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

Copy link
Copy Markdown

Deploying drive-web with  Cloudflare Pages  Cloudflare Pages

Latest commit: 1d8b90f
Status: ✅  Deploy successful!
Preview URL: https://73ff5c35.drive-web.pages.dev
Branch Preview URL: https://fix-remove-is-upload-allowed.drive-web.pages.dev

View logs

@CandelR CandelR left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Waiting the modal that warns the user that the owner hasn't enough space :)

@xabg2

xabg2 commented Apr 15, 2026

Copy link
Copy Markdown
Contributor Author

The reached plan dialog has been modified for shared folders. See the attached image to see how it looks now for shared folders.
Captura de pantalla 2026-04-15 a las 10 30 35

@xabg2
xabg2 requested a review from CandelR April 15, 2026 09:17
CandelR
CandelR previously approved these changes Apr 15, 2026
@xabg2
xabg2 requested a review from a team as a code owner April 15, 2026 14:38
@xabg2
xabg2 requested a review from CandelR April 15, 2026 14:40
@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

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

See analysis details on SonarQube Cloud

@CandelR
CandelR merged commit 407be1e into master Apr 27, 2026
10 of 11 checks passed
@CandelR
CandelR deleted the fix/remove-is-upload-allowed-for-shared-uploads branch April 27, 2026 07:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants