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.

Sign up for GitHub

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

Jump to bottom

tell: check if a tellee is a valid identifier #2583

Open
SnoopJ opened this issue Dec 11, 2023 · 0 comments
Open

tell: check if a tellee is a valid identifier #2583

SnoopJ opened this issue Dec 11, 2023 · 0 comments
Labels
Bug Things to squish; generally used for issues Needs Triage Issues that need to be reviewed and categorized

Comments

@SnoopJ
Copy link
Contributor

SnoopJ commented Dec 11, 2023

Requested Feature

Filing as follow-up from #2582.

Currently, the tell plugin does no verification that a tellee is a valid identifier, but it's something we may want to consider for longer term purposes. Example cases where we will store reminders for a nick we will never see:

22:49 <+SnowJ> .tell 猫 nyaaa
22:49 <+Sopel> SnowJ: I'll pass that on when 猫 is around.
22:50 <+SnowJ> .tell shinji@NERV get in the robot
22:50 <+Sopel> SnowJ: I'll pass that on when shinji@NERV is around.

Problems Solved

No response

Alternatives

No response

Notes

No response
@SnoopJ SnoopJ added the Bugfix Generally, PRs that reference (and fix) one or more issue(s) label Dec 11, 2023
@SnoopJ SnoopJ mentioned this issue Dec 11, 2023
Merged
4 tasks
@dgw dgw added Bug Things to squish; generally used for issues Needs Triage Issues that need to be reviewed and categorized and removed Bugfix Generally, PRs that reference (and fix) one or more issue(s) labels Dec 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug Things to squish; generally used for issues Needs Triage Issues that need to be reviewed and categorized
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@dgw @SnoopJ