-
Notifications
You must be signed in to change notification settings - Fork 23
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
Self timeout command #85
base: master
Are you sure you want to change the base?
Conversation
@@ -64,6 +66,35 @@ async def sync(self, ctx): | |||
async def eight_ball(self, ctx): | |||
await ctx.send(random.choice(self.eight_ball_options)) | |||
|
|||
@commands.command(name="zipit") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
suggestion: name this "lockin" instead
value = int(match.group(1)) * v["duration"] | ||
total_duration += datetime.timedelta(minutes=value) | ||
|
||
if total_duration.seconds == 0: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe set an upper bound to like 2w? to protect users
Why not add this command as a slash command? |
I have a personal bias against slash commands. But it's a good point, should probably be a slash command. There's no need for speed which is necessary for commands like snipe. |
I didn’t really think about speed. For me, it was more about the convenience that slash cmds provide. I think we should migrate cmds which don’t have to be fast to slash cmds, and make new ones slash cmds as well. |
?zipit 3d3h
to time out yourself for 3 days 3 hoursIdea by @Mattwmaster58