-
Notifications
You must be signed in to change notification settings - Fork 17
Conversation
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.
session isn’t a attribute of our current bot class. And target isn’t a valid name in the command. Revise and commit again.
url="https://api.waifu.pics/sfw/slap" | ||
) as resp: | ||
await ctx.send( | ||
embed=discord.Embed( |
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.
Should be disnake.Embed(...)
to maintain consistency
await ctx.send(embed=embed2) | ||
async def slap(self, ctx, member: Member = None): | ||
"""slap someone""" | ||
async with self.bot.session.get( |
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.
The bot instance doesn't have a session variable like @Yat-o pointed out, though we are working on reusing the aiohttp.ClientSession()
instance.
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.
Please resolve the following issues I have outlined in the comments.
Denied PR because of lack of attention, i suggest finishing on PR before opening a new one. |
No description provided.