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

added the reload extension command #285

Merged
merged 4 commits into from
Oct 9, 2024
Merged

Conversation

iyad-f
Copy link
Contributor

@iyad-f iyad-f commented Oct 8, 2024

  • cogs.admin has been added
  • reloadext command has been added to cogs.admin

@hitblast hitblast linked an issue Oct 8, 2024 that may be closed by this pull request
@hitblast hitblast added the hacktoberfest-accepted Pull requests approved for Hacktoberfest label Oct 8, 2024
Copy link
Member

@hitblast hitblast left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Needs some changes I guess.

core/bot.py Outdated Show resolved Hide resolved
Copy link
Member

@hitblast hitblast left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@hitblast hitblast merged commit 7576784 into IgKniteDev:main Oct 9, 2024
3 checks passed
@hitblast hitblast added commands Addition, modification or deletion of commands feat New features labels Oct 9, 2024
@iyad-f iyad-f deleted the reloadext branch October 9, 2024 03:23


class Admin(commands.Cog):
@commands.command(name='reloadext')

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hey @hitblast this should be @commands.slash_commands right?
@iyad-f , Can you provide a example of how to use this command. I am not able to use this command. Wanted to know if there is any other way of using command than "slash"

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it is not going to be a slash command since we don't wanna go with the overhead of registring it to just one guild where admins is present, we rather go with a prefixed command which only owners of the bot will be able to use. As for the second part of the question how to use this, its simple ?reloadext <extiontion_path> e.g. ?reloadext cogs.admin

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
commands Addition, modification or deletion of commands feat New features hacktoberfest-accepted Pull requests approved for Hacktoberfest
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEATURE] reload extentsion command
3 participants