Skip to content
This repository has been archived by the owner on Jul 19, 2023. It is now read-only.

Implement part #20

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Implement part #20

wants to merge 3 commits into from

Conversation

tadzik
Copy link

@tadzik tadzik commented Oct 16, 2016

This relies on #19 being merged. It works perfectly fine as far as the matrix side of things is concerned, though when I tried it on weechat it didn't automatically close the chat buffer; not sure if that's weechat's fault or does some feedback irc message needs to be sent back to the user? Also, RoomLeaveInput and RoomLeaveResponse exist only to satisfy the do_json_post's API; perhaps it'd be better to either teach do_json_post to deal with empty payloads, or create a dummy struct for those? What do you think?

@erikjohnston
Copy link
Member

This needs to be updated to do a similar thing as the join here, in terms of sending a PART message down the IRC connection when we get a 200

@rodneyrod
Copy link

Any ETA on when this could get merged?

@jplatte
Copy link
Contributor

jplatte commented Feb 27, 2020

@tadzik Are you still interested in getting this functionality into matrix-ircd?

@tadzik
Copy link
Author

tadzik commented Feb 27, 2020

@jplatte I'm planning to sit back to down to this in near future once $work slows down a little, but I don't have any ETA. If you wanna follow up on this issue/patch, go right ahead :)

@jplatte
Copy link
Contributor

jplatte commented Feb 27, 2020

I don't have the time myself right now, just doing triage :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants