Skip to content

test: Run tests against nextcloud/server#57760 share-update-check-single#16987

Open
nickvergessen wants to merge 2 commits intomainfrom
test/server-57760/share-update-check-single
Open

test: Run tests against nextcloud/server#57760 share-update-check-single#16987
nickvergessen wants to merge 2 commits intomainfrom
test/server-57760/share-update-check-single

Conversation

@nickvergessen
Copy link
Member

@nickvergessen nickvergessen commented Feb 5, 2026 •
edited
Loading

  • Root cause seems to be that for a share created in a new conversation, the other participants aren't registered as being in the room yet by the time the ShareCreatedEvent is emitted. So fetching the shares for the user doesn't return the share in the new room.
  • fix: only validate mounts for new share server#57760 seems to fix it for the basic 1-1 room at least. Not sure if there are other edge cases

@nickvergessen
test: Run tests against nextcloud/server#57760 share-update-check-single
4dee540 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
test: Fix assumption that files are in Talk/ afterwards
31c83d9 
Signed-off-by: Joas Schilling <coding@schilljs.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@nickvergessen