tuna / fishroom

Message forwarding for multiple IM protocols
GNU General Public License v3.0
288 stars 56 forks source link

Add preliminary support for Matrix #56

Closed mytbk closed 7 years ago

bigeagle commented 7 years ago

wow, cooool! Hmmm, have you tested it?

mytbk commented 7 years ago

The test() in matrix.py has not been tested yet. I've tested it with a Matrix room together with XMPP and IRC.

Multiple bindings has been tested with:

    'bindings': {
        "archlinux-cn": {
            "xmpp": 'archlinux@conference.cnjabber.net',
            "matrix": '#archlinux:matrixim.cc'
        },

        "linuxmasterrace": {
            "matrix": '#linuxmasterrace:matrix.org',
            "xmpp": 'linuxmasterrace@conference.cnjabber.net'
        }
    }

Only m.text msgtype in m.room.message event (https://matrix.org/docs/spec/client_server/r0.2.0.html#m-room-message-msgtypes) is supported.

Nick is not supported.

bigeagle commented 7 years ago

OK. I've commented on the code, you can fix the little problems that I've pointed out. I'll merge this as soon as you push the updates.

mytbk commented 7 years ago

I've added nick support in another commit.

bigeagle: I'm sorry, but I can't see where your comment is.

bigeagle commented 7 years ago

I'm sorry I forgot to submit my comments. :cry: