Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

The upload file button will be disabled after reaching the limit 250MB #3219

Closed
wants to merge 2 commits into from

Conversation

Noorain464
Copy link

Fixes #issue-number: Error message claiming size limit of 250MB reached #2080
Changes:

I have verified that this pull request:

  • has no linting errors (npm run lint)
  • has no test errors (npm run test)
  • is from a uniquely-named feature branch and is up to date with the develop branch.
  • is descriptively named and links to an issue number, i.e. Fixes #123

@@ -11,6 +11,7 @@ import DownArrowIcon from '../../../images/down-filled-triangle.svg';
import FolderRightIcon from '../../../images/triangle-arrow-right.svg';
import FolderDownIcon from '../../../images/triangle-arrow-down.svg';
import FileTypeIcon from './FileTypeIcon';
import { sizeLimit, currentSize } from './AssetSize';
Copy link
Collaborator

Choose a reason for hiding this comment

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

This doesn't seem to import the correct constants you've created in AssetSize.jsx!

>
{t('FileNode.UploadFile')}
</button>
{currentSize < sizeLimit ? (
Copy link
Collaborator

Choose a reason for hiding this comment

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

Maybe this could be handled by changing the css property depending on currentSize < sizeLimit rather than the whole button?

@raclim
Copy link
Collaborator

raclim commented Aug 12, 2024

Thanks so much for your work on this!

After reviewing this, it seems like you're trying to add an alert message once a user has reached the 250MB limit for storing assets. We do have error handling for this implemented already in the File Upload Modal (UploadFileModal.jsx), although I do think it could be interesting to explore using an alert instead or doing this in the sidebar navigation!

I don't think the overall changes here are addressing the original issue, but I've left a few notes on what's here so far! I feel that the original issue seems to involve how the asset sizes are being determined, so it might be better to try working from there instead. I'm going to close this pull request for now, but please feel free to revisit this with any updates!

@raclim raclim closed this Aug 12, 2024
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