-
-
Notifications
You must be signed in to change notification settings - Fork 40
Purge messages command #110
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
Conversation
doomed-neko
left a comment
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.
passed the vibe check for me
|
OK, let me get to work on those. |
update purge.py
electron271
left a comment
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.
idealy these comments should be fixed but honestly it doesnt matter that much right now (idealy they should though)
im going to leave it up to @kzndotsh if i should merge or not with those two issues
|
let me fix these issues, then ill be okay with merging. i dont want to merge something if it still has issues. |
Update purge.py to grab user ids from constants.py
Adding a command to purge messages.