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

Remove upload_limit in files app as it is not needed with PUT upload #1511

Merged
merged 1 commit into from
Nov 2, 2016

Conversation

LukasReschke
Copy link
Member

@LukasReschke LukasReschke commented Sep 26, 2016

From owncloud/core#26069

The web UI now uses for PUT uploads which aren't restricted by PHP's
upload_max_filesize and post_max_size

Signed-off-by: Lukas Reschke [email protected]

@LukasReschke LukasReschke added 3. to review Waiting for reviews downstream labels Sep 26, 2016
@LukasReschke LukasReschke added this to the Nextcloud 11.0 milestone Sep 26, 2016
@mention-bot
Copy link

@LukasReschke, thanks for your PR! By analyzing the annotation information on this pull request, we identified @ogasser, @butonic and @MorrisJobke to be potential reviewers

@LukasReschke LukasReschke mentioned this pull request Sep 26, 2016
4 tasks
@LukasReschke LukasReschke added 1. to develop Accepted and waiting to be taken care of and removed 3. to review Waiting for reviews labels Oct 6, 2016
…(#26069)

The web UI now uses for PUT uploads which aren't restricted by PHP's
upload_max_filesize and post_max_size

Signed-off-by: Lukas Reschke <[email protected]>
@rullzer rullzer force-pushed the master-kill-upload-limit branch from 4b8837b to 7e70150 Compare November 2, 2016 19:43
@rullzer
Copy link
Member

rullzer commented Nov 2, 2016

#1283 is in.
Rebased.

LGTM

@rullzer rullzer added 3. to review Waiting for reviews and removed 1. to develop Accepted and waiting to be taken care of labels Nov 2, 2016
@rullzer
Copy link
Member

rullzer commented Nov 2, 2016

@MorrisJobke
Copy link
Member

Tested and works 👍

@MorrisJobke MorrisJobke added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Nov 2, 2016
@rullzer rullzer merged commit a8d9bf4 into master Nov 2, 2016
@rullzer rullzer deleted the master-kill-upload-limit branch November 2, 2016 20:24
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants