Show name for bridged messages

[gary-kim]

Requires 42wim/matterbridge#1506. Now merged!

The way the current iteration works is that messages received from the bridge-bot user is labeled as Actor Type bridged. Actor Type bridged messages store the message author name in the Actor Id field in oc_comments. When serving to the frontend, this is slightly changed to put what is in Actor Id into Actor Display Name and set the Actor Id to bridge-bot (the reason it was done this way was to simplify the changes that would need to be made to clients).

As far as clients have to care, the only difference is the addition of the Actor Type bridged and that for bridged messages, the Actor Display Name can be different for every message.

Screenshots

[nickvergessen]

Requires 42wim/matterbridge#1506

I guess we have to wait until that is merged?

[gary-kim]

Yup. It'd be noop if merged right now

[nickvergessen]

noop in terms of "works as before" or in terms of "breaks and doesn't do anything anymore"

[gary-kim]

Works as before

[gary-kim]

If everything seems fine on this side, we could merge now. It should does (tested) function the same as before with a Matterbridge version that does not support the SeperateDisplayName config.

[nickvergessen]

Maybe you can ping them again?

[gary-kim]

42wim/matterbridge#1506 is now merged so this feature should now work with the executables here: https://github.com/42wim/matterbridge/actions/runs/953056470.

[nickvergessen]

Good from my POV, but I guess we should wait for 22 to go out as the mobile apps need a patch too

[gary-kim]

Would have been nice to get it into 22 but we're lacking in time, I suppose

[nickvergessen]

Android patch at nextcloud/talk-android#1355
iOS ticket raised in nextcloud/talk-ios#579 but at least it shows something already.

[nickvergessen]

Can you rebase @gary-kim