Skip to content

Revert "feat(bulk-upload): change the default to disabled as there are still some bugs present" - #52137

Merged
artonge merged 1 commit into
masterfrom
revert-51431-enh/noid/disable-bulk-upload
Apr 14, 2025
Merged

Revert "feat(bulk-upload): change the default to disabled as there are still some bugs present"#52137
artonge merged 1 commit into
masterfrom
revert-51431-enh/noid/disable-bulk-upload

Conversation

@artonge

@artongeartonge commented Apr 11, 2025

Copy link
Copy Markdown
Collaborator

Reverts #51431

@artonge
artonge requested a review from a team as a code ownerApril 11, 2025 21:39
@artonge
artonge requested review from ArtificialOwl, come-nc and skjnldsv and removed request for a teamApril 11, 2025 21:39
szaimen
szaimen previously requested changes Apr 11, 2025
View reviewed changes

@szaimenszaimen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Blocking for now.

@Rello needs to decide how to move forward with this topic.

@artonge

artonge commented Apr 14, 2025

Copy link
Copy Markdown
CollaboratorAuthor

Not really for @Rello to decide...

@artonge
artonge dismissed szaimen’s stale reviewApril 14, 2025 07:27

This is not up to the clients team

szaimen
szaimen previously requested changes Apr 14, 2025
View reviewed changes

@szaimenszaimen left a comment •
edited
Loading

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This is not up to the clients team

Lets discuss this in a call before we move forward with this

@szaimen

szaimen commented Apr 14, 2025

Copy link
Copy Markdown
Contributor

This is not up to the clients team

Lets discuss this in a call before we move forward with this

https://cloud.nextcloud.com/call/7187151879

@susnuxsusnux added enhancement 3. to review Waiting for reviews pending documentation This pull request needs an associated documentation update labels Apr 14, 2025
@artonge
artonge merged commit c74e4ad into masterApr 14, 2025
@artonge
artonge deleted the revert-51431-enh/noid/disable-bulk-upload branch April 14, 2025 09:41
@szaimenszaimen added this to the Nextcloud 32 milestone Apr 14, 2025
@nextcloud-botnextcloud-bot mentioned this pull request Aug 19, 2025
Merged
@joshtrichardsjoshtrichards added the hotspot: file transfer performance upload & download performance related optimizations label Aug 3, 2026
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@susnuxsusnuxsusnux approved these changes
@provokateurinprovokateurinprovokateurin approved these changes
@szaimenszaimenszaimen left review comments
@ArtificialOwlArtificialOwlAwaiting requested review from ArtificialOwlArtificialOwl is a code owner automatically assigned from nextcloud/server-backend
@skjnldsvskjnldsvAwaiting requested review from skjnldsvskjnldsv is a code owner automatically assigned from nextcloud/server-backend
@come-nccome-ncAwaiting requested review from come-nccome-nc is a code owner automatically assigned from nextcloud/server-backend

Assignees

No one assigned

Labels

3. to reviewWaiting for reviewsenhancementhotspot: file transfer performanceupload & download performance related optimizationspending documentationThis pull request needs an associated documentation update

Projects

None yet

Milestone

Nextcloud 32

Development

Successfully merging this pull request may close these issues.

7 participants

@artonge@szaimen@susnux@provokateurin@joshtrichards@tobiasKaminsky@skjnldsv