Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] Rewrite receipt handling #3873

Closed
wants to merge 33 commits into from
Closed
Changes from 1 commit
Commits
Show all changes
33 commits
Select commit Hold shift + click to select a range
6e9a3c4
Ignore receipts pointing at missing or invalid events
andybalaam Oct 20, 2023
5cfe82e
Remove extra whitespace from log message
andybalaam Oct 31, 2023
f542315
Unit tests for ignoring invalid receipts
andybalaam Oct 31, 2023
9fa5374
Improve comments around getEventReadUpTo
andybalaam Nov 1, 2023
9060e0e
Re-instate second param to compareEventOrdering in test
andybalaam Nov 1, 2023
3686d61
Further improve comments around getEventReadUpTo
andybalaam Nov 1, 2023
4ee0fba
WIP
andybalaam Nov 3, 2023
ad79e97
Fix typo in thread test
andybalaam Nov 7, 2023
43ad085
Merge branch 'develop' into andybalaam/rewrite-receipts
andybalaam Nov 9, 2023
5bd2444
Merge branch 'develop' into andybalaam/rewrite-receipts
andybalaam Nov 10, 2023
26598e1
Remove extraneous import
andybalaam Nov 10, 2023
2bf53df
First ideas of tests for RoomReceipts
andybalaam Nov 10, 2023
c6b0a89
More tests for RoomReceipts
andybalaam Nov 10, 2023
684b831
More tests and a fix because unit tests rule
andybalaam Nov 10, 2023
045e169
Add a test for mixing threaded and unthreaded receipts
andybalaam Nov 10, 2023
fce2d98
Break out user-sent-last-message logic into a function
andybalaam Nov 10, 2023
b61bc69
Fix incorrect failing test
andybalaam Nov 10, 2023
b88706e
Enhance mock to allow tests to pass with new cod
andybalaam Nov 10, 2023
4c04a7b
Add a test for two events in old timelines
andybalaam Nov 13, 2023
d1cb13d
Add a test for events received after we sent one
andybalaam Nov 13, 2023
037b6b7
Merge branch 'develop' into andybalaam/rewrite-receipts
andybalaam Nov 13, 2023
5e7dfd3
Rename compare-event-ordering
andybalaam Nov 13, 2023
b236ebb
Merge branch 'develop' into andybalaam/rewrite-receipts
andybalaam Nov 13, 2023
7aa570e
Make compareEventOrdering use -1, 0 or 1 (or null) exclusively
andybalaam Nov 13, 2023
59f3214
Re-instate methods I deleted from public interfaces
andybalaam Nov 13, 2023
7452baa
Update comment to remove unexpected case
andybalaam Nov 13, 2023
f6d9cd8
Improve comments about code to delete
andybalaam Nov 13, 2023
2d10141
Merge branch 'develop' into andybalaam/rewrite-receipts
andybalaam Nov 13, 2023
5a0fbbd
Merge branch 'develop' into andybalaam/rewrite-receipts
andybalaam Nov 15, 2023
56875a7
BREAKS UNIT TESTS use correct thread when synthesizing receipts
andybalaam Nov 16, 2023
5b355d5
Merge branch 'develop' into andybalaam/rewrite-receipts
andybalaam Nov 16, 2023
0861f6b
Fix incorrect thread ID in read-receipt tests
andybalaam Nov 17, 2023
612e35e
Fix unit tests to reflect the fix putting synthetic receipts in the r…
andybalaam Nov 17, 2023
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Merge branch 'develop' into andybalaam/rewrite-receipts
andybalaam committed Nov 13, 2023
commit b236ebb9f4c47c36744d52b3c8c727c9529ce04b

This merge commit was added into this branch cleanly.

There are no new changes to show, but you can still view the diff.