Kevin Boos

> `original` may macthing multi images > > `file` still may macthing multi images > > But one `MediaRequestParameters` must matching one image I don't understand what this means.

I've marked this as waiting-on-author since I have a lot of questions that need to be answered before I do another review round.

> @kevinaboos See [aaravlu#2](https://github.com/aaravlu/robrix/pull/2) and it's diff to know why we cant set_image_keys when occupied: > > it's a test based on branch fix327-2 huh? I don't understand, and I...

At this point, I think it's best if we move on from this debate, as it has been ongoing for far too long. I will gladly accept your contributions for...

this is blocked on me, so i'll add the `waiting-on-review` label to remind myself. Will be a while before I get to it, though.

This is still waiting on me, so let's leave the `waiting-on-review` label in place. However, it'll be a while before I can get to it, probably early June.

This has already been self-assigned to me, but sure, you can take a look at it. > Our intention is to change `Vec` to `HashMap`, right? yep. I wouldn't be...

what PR are you referring to? Also we don't have any checks, I removed the old one but will add some basic ones later.

This is blocked on the Matrix Rust SDK, which currently doesn't fully implement EventCache features that allow the client to customize how events are extracted from and imported into the...

Hmm, actually it appears that the SDK might support this, at least in part, by using the `ClientBuilder` to configure a client to use an existing state store and cache...