Skip to content

Conversation

@juliusknorr
Copy link
Member

Different attempt to #23294 to check the available space on a subdirectory when uploading a file by dragging it to the browser.

Copy link
Member

@skjnldsv skjnldsv left a comment

Choose a reason for hiding this comment

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

Code make sense! Minimal legacy code changes too! 👍

@juliusknorr juliusknorr added 2. developing Work in progress and removed 3. to review Waiting for reviews labels Nov 16, 2020
Copy link
Member

@ChristophWurst ChristophWurst left a comment

Choose a reason for hiding this comment

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

Tested and works. I can upload a file to an external sub directory while the user's root directory is full 👍

@ChristophWurst ChristophWurst force-pushed the bugfix/noid/quota-upload branch from 2f0de61 to 0c19826 Compare November 17, 2020 08:31
Signed-off-by: Julius Härtl <[email protected]>
Signed-off-by: Christoph Wurst <[email protected]>
@ChristophWurst ChristophWurst force-pushed the bugfix/noid/quota-upload branch from 0c19826 to f9536b0 Compare November 17, 2020 08:44
mtime: parseInt($el.attr('data-mtime'), 10),
type: $el.attr('data-type'),
etag: $el.attr('data-etag'),
quotaAvailableBytes: $el.attr('data-quota'),
Copy link
Member

Choose a reason for hiding this comment

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

this will also be set for files, but I guess it's fine

Copy link
Member

@ChristophWurst ChristophWurst left a comment

Choose a reason for hiding this comment

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

Still works after minor modifications :)

@ChristophWurst ChristophWurst added 4. to release Ready to be released and/or waiting for tests to finish and removed 2. developing Work in progress labels Nov 17, 2020
@ChristophWurst ChristophWurst added this to the Nextcloud 21 milestone Nov 17, 2020
@ChristophWurst
Copy link
Member

/backport to stable20

@ChristophWurst
Copy link
Member

/backport to stable19

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

Labels

4. to release Ready to be released and/or waiting for tests to finish bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants