matrix-org / matrix-bifrost

General purpose bridging with a variety of backends including libpurple and xmpp.js
Apache License 2.0
163 stars 31 forks source link

Better implement avatar handling for rooms and occupants. #274

Closed maranda closed 3 years ago

maranda commented 3 years ago

These changes were implemented to improve Bifrost for our usage on the aria-net.org server.

Currently it implements the following features that were missing:

Fix a few bugs and issues:

maranda commented 3 years ago

This seems to be overlapping slightly with #272, so I'd rather remove these changes here, since they have their own specialized PR and all.

That's fine but I'm not sure how to bisect those changes from the rest without remaking a pull request 🤔

Half-Shot commented 3 years ago

This seems to be overlapping slightly with #272, so I'd rather remove these changes here, since they have their own specialized PR and all.

That's fine but I'm not sure how to bisect those changes from the rest without remaking a pull request thinking

I've since merged those changes, so you just need to resolve the merge and I think we can re-review from there.

maranda commented 3 years ago

Will rebase the pull request, this is becoming a mess to mantain with all syncs and is unreadable.