Skip to content

fix: Only handle recent awareness messagess from sessions - #4053

Merged
max-nextcloud merged 2 commits into
mainfrom
bugfix/noid/awareness-latest
Apr 17, 2023
Merged

fix: Only handle recent awareness messagess from sessions#4053
max-nextcloud merged 2 commits into
mainfrom
bugfix/noid/awareness-latest

Conversation

@juliusknorr

@juliusknorrjuliusknorr commented Apr 14, 2023

Copy link
Copy Markdown
Member

fixes#4010

@juliusknorr
juliusknorrforce-pushed the bugfix/noid/awareness-latest branch from e9f4545 to 94e3b5eCompareApril 14, 2023 11:35
@juliusknorrjuliusknorr added bug Something isn't working 3. to review labels Apr 14, 2023
@juliusknorrjuliusknorr added this to the Nextcloud 27 milestone Apr 14, 2023
@juliusknorr

Copy link
Copy Markdown
MemberAuthor

/backport to stable26

@cypress

cypressBot commented Apr 14, 2023

Copy link
Copy Markdown

1 flaky tests on run #9425 ↗︎

014210Flakiness1

Details:

fix: Only handle recent awareness messagess from sessions
Project: TextCommit:f6c1e71a90
Status: PassedDuration: 04:10 💡
Started: Apr 17, 2023 7:07 AMEnded: Apr 17, 2023 7:11 AM
Flakiness cypress/e2e/sync.spec.js • 1 flaky test

ViewOutputVideo

Test Artifacts
Sync > saves on close OutputScreenshots

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

@max-nextcloud
max-nextcloudforce-pushed the bugfix/noid/awareness-latest branch from 94e3b5e to 298f558CompareApril 15, 2023 07:04

@max-nextcloudmax-nextcloud left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Makes sense to me. I'm wondering if we should only use the last awareness message but i need to dig into the awareness protocol more for that.
This is definitely a step in the right direction. So let's get it in.

Signed-off-by: Julius Härtl <jus@bitgrid.net>
@max-nextcloud
max-nextcloudforce-pushed the bugfix/noid/awareness-latest branch from 298f558 to 807b47fCompareApril 17, 2023 06:43
@max-nextcloud

Copy link
Copy Markdown
Collaborator

/compile

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewbugSomething isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

NC26: author tags are shown although no other user is in the file

3 participants

@juliusknorr@max-nextcloud@nextcloud-command