Page MenuHomePhorge

Consider local read marker when checking whether a room is read
ClosedPublic

Authored by tusooa on Sat, Nov 16, 5:58 PM.
Tags
None
Referenced Files
F112184: D205.1732273802.diff
Thu, Nov 21, 3:10 AM
F112183: D205.1732273782.diff
Thu, Nov 21, 3:09 AM
F112182: D205.1732273720.diff
Thu, Nov 21, 3:08 AM
F112122: D205.1732271816.diff
Thu, Nov 21, 2:36 AM
F112094: D205.1732271190.diff
Thu, Nov 21, 2:26 AM
F112088: D205.1732268171.diff
Thu, Nov 21, 1:36 AM
F111984: D205.1732246600.diff
Wed, Nov 20, 7:36 PM
F111983: D205.1732246550.diff
Wed, Nov 20, 7:35 PM
Subscribers
None

Details

Summary

This makes kazv consider an event as read when either the local read marker or the remote read receipt from sync has it.

Requires: https://iron.lily-is.land/D204

Type: fix

BUG: https://iron.lily-is.land/T113

Test Plan

Send from another session a message that generates a notification.
Open kazv to receive the message and the notification but do not mark it as read.
Reopen kazv with an invalid proxy (https_proxy=socks5://localhost:12345) such that you cannot make any internet requests, and switch to that room.
Verify the room is marked as read.
Reopen kazv removing the proxy, and switch to that room again. Verify a request is sent to the server updating the read receipt.

Diff Detail

Repository
rK kazv
Branch
tusooa/stacked
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 329
Build 644: GitLab CI for kazv
Build 643: arc lint + arc unit

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Sat, Nov 16, 6:19 PM
Harbormaster failed remote builds in B329: Diff 544!
This revision was not accepted when it landed; it landed in state Changes Planned.Sun, Nov 17, 7:43 AM
This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.