-
Notifications
You must be signed in to change notification settings - Fork 301
Fix #714: large file upload tuning as configuration #716
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
Fix #714: large file upload tuning as configuration #716
Conversation
Signed-off-by: WrenIX <dev.github@wrenix.eu>
Signed-off-by: WrenIX <dev.github@wrenix.eu>
Signed-off-by: WrenIX <dev.github@wrenix.eu>
Signed-off-by: WrenIX <dev.github@wrenix.eu>
Signed-off-by: WrenIX <dev.github@wrenix.eu>
Signed-off-by: antoinetran <antoinetran@users.noreply.github.com>
Signed-off-by: antoinetran <antoinetran@users.noreply.github.com>
a554746 to
e9bad15
Compare
|
Hi @wrenix , I do not understand if this Pull Request has been closed because it is merged to nextcloud main branch? However it seems it is not yet in the branch. For example: https://github.com/nextcloud/helm/blob/main/charts/nextcloud/values.yaml does not contains the custom block. |
|
Github is saying there are unmerged commits here. |
|
because the |
|
Ohhhhh ok, I thought you merged it, but I couldn't find the result in main branch. Ok, recreating it. |
|
New pull request created. |
Description of the change
Fix #714
Benefits
Can now upload large file. And if the default values are not enough, can override them in
values.yaml.Possible drawbacks
Applicable issues
Additional information
Checklist
Chart.yamlaccording to semver.