matrix-appservice-discord

matrix-appservice-discord copied to clipboard

Add the News/Announcement channel type to the list of allowed channel types to bridge between. This can be useful for administrators to centralize announcements between services to reduce the ammount of busy-work that they may need to do.

This fixes #774 and it fixes #813, which is a duplicate

May be useful to add a test.

I'm not sure how to add a test for this kind of stuff, eheh. Most of the code I deal with are core python services that don't exactly rely on external services :sweat_smile:

Always willing to learn, I just don't know where to start for something out of my field of expertise.

Not that familiar either, but you probably want to make a copy of this test here: https://github.com/matrix-org/matrix-appservice-discord/blob/abf295045007a2504acff96e56a330e7e5ce0a16/test/test_matrixeventprocessor.ts#L255 And then check whether non-announcement and announcement messages in news channel work.

Thank you for pointing me in the right direction! Got a test added

Are there any blockers on this PR? Its quite badly needed.