Skip to content

Conversation

@backportbot
Copy link

@backportbot backportbot bot commented Mar 25, 2025

@backportbot backportbot bot added the 3. to review Waiting for reviews label Mar 25, 2025
@backportbot backportbot bot added this to the Nextcloud 31.0.3 milestone Mar 25, 2025
come-nc added 9 commits March 27, 2025 12:18
Even if dav endpoint is now used.

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Remove duplicate activity publishing from share controller download,
 listen to BeforeZipCreatedEvent to publish activity for folders, and
 cache folders activity to avoid sending activity for each file in the
 folder.

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
…e file

Avoids flooding activities when someone browse a video in the web player.

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
…the request

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
…urrency issues

If several people are watching and seeking the same video file we do not
 want the cache key to be the same or it would flood activity again.

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Fixes "Object for placeholder file is invalid, value 21 for key id is not a string"
Exception spotted in tests logs.

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
@come-nc come-nc force-pushed the backport/51458/stable31 branch from 06dd8e6 to 304112b Compare March 27, 2025 11:20
@come-nc come-nc marked this pull request as ready for review March 27, 2025 11:20
@come-nc come-nc enabled auto-merge March 27, 2025 17:19
@come-nc come-nc merged commit 964a59a into stable31 Mar 27, 2025
215 of 228 checks passed
@come-nc come-nc deleted the backport/51458/stable31 branch March 27, 2025 17:24
@Altahrim Altahrim mentioned this pull request Apr 3, 2025
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants