-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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
[stable23] Disable bulk upload by not advertising it #31171
Conversation
It has been reported broken on many instances. Disable it for now until it can be fixed. Signed-off-by: Vincent Petry <vincent@nextcloud.com>
as stated in nextcloud/desktop#4243 I had this issue also on my own server and commenting out the capability (and restarting the client) solved the issue for me |
Doesn't it make more sense to put the feature behind a feature flag in the desktop? If we disable it, we can't even test it in real use cases. |
Then all desktop clients would need an upgrade first. I think that would take longer. Short before the release, the best is to disable it for now and then go for a good solution that may involve the clients, too, and give some time to upgrade without this experience. |
restarted the build, too many weird fails due to env issues |
I confirm that disabling bulk upload on the server and restarting the clients fixes the problem. Thank you very much for this solution. |
Signed-off-by: Joas Schilling <coding@schilljs.com>
also to master / 24: #31736 |
It has been reported broken on many instances.
Disable it for now until it can be fixed.
@matthieugallien FYI
to be put into the 23.0.3 hotfix release, to avoid more surprises for now