[adminutils] membersteal#47
[adminutils] membersteal#47phenom4n4n wants to merge 36 commits intofixator10:V3from phenom4n4n:emoji_steal
Conversation
Co-authored-by: Fixator10 <fixator10@gmail.com>
Co-authored-by: Fixator10 <fixator10@gmail.com>
Co-authored-by: Fixator10 <fixator10@gmail.com>
Co-authored-by: Fixator10 <fixator10@gmail.com>
Co-authored-by: Fixator10 <fixator10@gmail.com>
Co-authored-by: Fixator10 <fixator10@gmail.com>
fixator10
left a comment
fixator10
left a comment
There was a problem hiding this comment.
About that 'name' arg... What about implementing argparse for that?
Example: https://github.com/fixator10/Fixator10-Cogs/blob/V3.leveler_abc/leveler/commands/top.py
I don't know much about argparse and am not motivated to look into it right now. |
fixator10
left a comment
fixator10
left a comment
There was a problem hiding this comment.
Perhaps then we can add an special name _ to use original emoji's name?
like [p]emoji import _ 158155109289003029 will import emoji from 158155109289003029 with its original name
Co-authored-by: Fixator10 <fixator10@gmail.com>
Co-authored-by: Fixator10 <fixator10@gmail.com>
Co-authored-by: Fixator10 <fixator10@gmail.com>
Co-authored-by: Fixator10 <fixator10@gmail.com>
|
This pull request introduces 1 alert when merging fe2b411 into 27ee04a - view on LGTM.com new alerts:
|
| except discord.HTTPException as e: | ||
| await ctx.send(chat.error(_("An error occured on adding an emoji: {}").format(e))) | ||
| else: | ||
| msg = _("{em} created with the name `{em.name}`".format(em=em)) |
There was a problem hiding this comment.
| msg = _("{em} created with the name `{em.name}`".format(em=em)) | |
| msg = _("{em} created with the name `{em.name}`".format(em=e)) |
NameError: name 'em' is not defined
| return | ||
| try: | ||
| await emoji.edit( | ||
| em = await emoji.edit( |
There was a problem hiding this comment.
Seems like emoji.edit doesnt return anything, you need to re-get emoji, e.g. via em = bot.get_emoji(emoji.id)
https://discordpy.readthedocs.io/en/stable/api.html#discord.Emoji.edit
|
This pull request introduces 1 alert when merging f0eeb16 into 1a28ef0 - view on LGTM.com new alerts:
|
|
This pull request introduces 1 alert when merging 88bcd45 into fd6c8e3 - view on LGTM.com new alerts:
|
2 participants
Works similar to the message steal command, but allows you to steal an emoji from a member's status.