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

[IMP] Better versality for new and edit commands #22

Merged
merged 1 commit into from
Mar 27, 2020

Conversation

arbaes
Copy link
Contributor

@arbaes arbaes commented Mar 27, 2020

none can be used as a bot command argument. What it changes:

new

edit

  • Allow editing/removing individually the message body, the embed's title and the embed's description

@eibex
Copy link
Owner

eibex commented Mar 27, 2020

Looks great, though I would add an explanation about these new arguments on line 178 for rl!new and line 304 (308 in the new code) for rl!edit.

Thanks a lot for all the help :)

EDIT: I will do that myself in the sqlite branch while I work on some other stuff.

@eibex eibex merged commit 9875482 into eibex:master Mar 27, 2020
eibex added a commit that referenced this pull request Mar 27, 2020
eibex pushed a commit that referenced this pull request Aug 23, 2020
eibex pushed a commit that referenced this pull request Aug 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants