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

Member, Channel and Server data classes #24

Open
itslukej opened this issue Oct 26, 2016 · 8 comments
Open

Member, Channel and Server data classes #24

itslukej opened this issue Oct 26, 2016 · 8 comments

Comments

@itslukej
Copy link
Owner

Example

>>> event.author.account
"eclipse"
>>> event.author.realname
"why does somebody not know how to flush the toilet"
>>> event.channel.topic
"zIRC  - IRC Protocol client library | https://github.com/itslukej/zirc | CHIPOTLE SOUTHWEST SAUCE"
>>> event.server.nicklen
16
@itslukej
Copy link
Owner Author

This may require a full lib rewrite, so I could probably do #6 at the same time.

@wolfy1339
Copy link
Collaborator

wolfy1339 commented Nov 1, 2016

Channel topic shouldn't be hard to implement, we just need to parse the TOPIC command, account is simple too we just need extended-join (to parse account on join), and multi-prefix

@wolfy1339
Copy link
Collaborator

Actually, we can use account-tag (IRCv3.2) to get the event author's account using message tags

@itslukej
Copy link
Owner Author

itslukej commented Nov 2, 2016

hmm.. Do popular IRC servers even support v3.2?

@wolfy1339
Copy link
Collaborator

freenode barely supports v3.1.
Any server that runs a fairly recent version of charybdis has IRCv3.2 support

@wolfy1339
Copy link
Collaborator

I was thinking about the server class, a lot if things we can already get from the MOTD

@BradleyShaw
Copy link
Contributor

Account tracking can be done with account-notify, extended-join and WHOX. Checking for WHOX support would require parsing/storing 005 so it would make sense to create the server class before trying to implement account tracking.

@wolfy1339
Copy link
Collaborator

wolfy1339 commented Jun 20, 2017

event.author should be incorporated into event.source

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

No branches or pull requests

3 participants