Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add LIST support #166

Open
Rixxan opened this issue Mar 27, 2022 · 0 comments · May be fixed by #172
Open

Add LIST support #166

Rixxan opened this issue Mar 27, 2022 · 0 comments · May be fixed by #172
Labels

Comments

@Rixxan
Copy link
Contributor

Rixxan commented Mar 27, 2022

Detailed Description

Support for the LIST function (RPL_LISTSTART, RPL_LIST, RPL_LISTEND) (RFC 1459 Section 4.2.6) would be a welcome addition to Pydle. This function would allow Pydle to be aware of all of the channels that exist in a given server or validate that a channel exists before taking an action.

Context

In our bot and use case, this allows the bot to be aware of channels that exist before taking an action, allowing us to filter actions that would reflect against a channel that does not exist (like forcing a user to join a channel or the bot attempting to join a channel that does not exist).

This would also add the ability to create checks on specific channels and get topics of a channel that exist, even if we are not in the channel already.

Possible Implementation

A partial implementation of this function has been done in our bot (HalpyBOT) here:

https://github.com/HullSeals/HalpyBOT/blob/58d68b8c55651ac4577af2749b86163b444c4f3e/halpybot/packages/ircclient/_listsupport.py

Your Environment

Rixxan added a commit to Rixxan/pydle that referenced this issue May 30, 2022
A basic lifting of the HalpyBOT List Handler (from @rik079) to Pydle
Rixxan added a commit to Rixxan/pydle that referenced this issue Jun 2, 2022
Allows users to call channel_list from the BasicClient, which returns a list containing the total LIST response from the server according to the IRC specification
@Rixxan Rixxan linked a pull request Jun 6, 2022 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants