Ivan Enderlin
Ivan Enderlin
See https://github.com/torrybr/matrix-rust-sdk/pull/1, which does most of the job.
@bnjbvr I've rebased on top of `main` but I've marked all the new commits as `!fixup`, except https://github.com/matrix-org/matrix-rust-sdk/pull/3585/commits/98aae99674def3e38505f71c464846de850237ba, https://github.com/matrix-org/matrix-rust-sdk/pull/3585/commits/50c804b25308fdadd7dfe7299eeaf9365ba0e8a3, https://github.com/matrix-org/matrix-rust-sdk/pull/3585/commits/235297849e3c086353354ca35ef84e962a608d5c, https://github.com/matrix-org/matrix-rust-sdk/pull/3585/commits/6132841f5c29e5984b663c46f7eac94cbb8f2a26, https://github.com/matrix-org/matrix-rust-sdk/pull/3585/commits/7bd042796647b080408d404dfebf09716c3e4d93, and https://github.com/matrix-org/matrix-rust-sdk/pull/3585/commits/824cb0682a072ed73a2c8f1c7d1b92b83ddf34c2.
I've changed the code to use the `timestamp` from the sliding sync room response instead of the `LatestEvent` origin server timestamp. Why? Because if we don't have a `LatestEvent`, then...
@bnjbvr One more time with https://github.com/matrix-org/matrix-rust-sdk/pull/3585/commits/1f1ff3dea1f332fa4777247c80768b9a7f019223 please 🥺?
Complement crypto is expected but I've disabled it. Force merging the PR as all other CI jobs are fine.
Lot to read in order to formulate a counter point of view. Give me few days :-). Thanks for your feedback! Edit: That's perfectly fine to close this PR in...
Thanks for reporting it. Yes, it's gonna change with our soon-to-start project about revamping the Latest Event API. Thanks!
About https://github.com/matrix-org/matrix-rust-sdk/pull/5274#issuecomment-3001669854 — It's great the last part is mentioned in the doc but the first 2/3 of the comment is also truly valuable! Can you add that on the...
You can rebase your fixup commits, then we can merge the PR.