YunoHost-Apps / mautrix_signal_ynh

Matrix signal package for YunoHost
GNU Affero General Public License v3.0
4 stars 10 forks source link

Support for Dendrite #69

Open Delegator9176 opened 1 year ago

Delegator9176 commented 1 year ago

I tried to install the app along with dendrite but it didn't work. Any chance that the Mautrix-Framework works with Dendrite, or what do I need to do to get it working?

Thanks in advance :)

MayeulC commented 1 year ago

Well, I think Dendrite is advanced enough to support it now.

That part of the bridge needs to be overhauled, so that a user may select any synapse or dendrite instance when installing the bridge (for now it asks the synapse instance number):

https://github.com/YunoHost-Apps/mautrix_signal_ynh/blob/350f877b57f67ac7d96676759ceda0b199db5a24/scripts/install#L40-L53

https://github.com/YunoHost-Apps/mautrix_signal_ynh/blob/350f877b57f67ac7d96676759ceda0b199db5a24/manifest.json#L32-L44

Ideally, we would also upgrade to the new manifest as well.

Then, registration should be adapted according to the docs:

https://github.com/YunoHost-Apps/mautrix_signal_ynh/blob/350f877b57f67ac7d96676759ceda0b199db5a24/scripts/install#L161-L168

All in all, not a lot of work, but a lot of testing, especially if we want to get this right and not pile another layer of hacks in the install script.

Delegator9176 commented 1 year ago

That would be great. As for testing, I'm happy to support that.