Ogiroid icon indicating copy to clipboard operation
Ogiroid copied to clipboard

added catfact

Open japlic opened this issue 1 year ago • 4 comments

I added a catfact command

  • [x] made it
  • [x] tested it

japlic avatar Apr 24 '24 22:04 japlic

Why did you remove the images from the other animals? Also this would be perfect as a new command not a replacement for the current animal command, since the command is for animal images.

TimWelm avatar Apr 25 '24 15:04 TimWelm

anyone

japlic avatar Apr 29 '24 00:04 japlic

He sorry for the delays

LevaniVashadze avatar Apr 29 '24 05:04 LevaniVashadze

Can you check the docs if a sub command of a sub command is possible so that the command is /animal cat fact ? Also I assume this is written by chatgpt based on your previous commits. So please test the code.

it is.

via this syntax

@bot.slash_command()
async def parent(inter):
    pass

@parent.sub_command_group()
async def group(inter):
    pass

@group.sub_command()
async def subcmd(inter):
    # Some stuff
    pass

JasonLovesDoggo avatar Apr 30 '24 17:04 JasonLovesDoggo

LGTM

JasonLovesDoggo avatar Jul 27 '24 01:07 JasonLovesDoggo

LGTM

Why did you merge it? We had requested changes

LevaniVashadze avatar Jul 27 '24 12:07 LevaniVashadze

LGTM

Why did you merge it? We had requested changes

Didn't see any relevance

JasonLovesDoggo avatar Jul 27 '24 13:07 JasonLovesDoggo