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

Make get_desynced_commands work with slash command groups #986

Merged

Conversation

Ratery
Copy link
Contributor

@Ratery Ratery commented Feb 10, 2022

Summary

Update get_desynced_commands function to be able to properly work with slash command groups.
This PR implements recursive slash commands traversing to compare all nested subcommands.

Checklist

  • If code changes were made then they have been tested.
    • I have updated the documentation to reflect the changes.
  • If type: ignore comments were used, a comment is also left explaining why
  • This PR fixes an issue.
  • This PR adds something new (e.g. new method or parameters).
  • This PR is a breaking change (e.g. methods or parameters removed/renamed)
  • This PR is not a code change (e.g. documentation, README, typehinting, examples, ...)

BobDotCom
BobDotCom previously approved these changes Feb 10, 2022
@BobDotCom BobDotCom enabled auto-merge (squash) February 10, 2022 20:34
EmmmaTech
EmmmaTech previously approved these changes Feb 10, 2022
@krittick
Copy link
Contributor

Please resolve conflicts. Thanks!

auto-merge was automatically disabled February 12, 2022 15:04

Head branch was pushed to by a user without write access

@Ratery Ratery dismissed stale reviews from EmmmaTech and BobDotCom via 3a882da February 12, 2022 15:04
@BobDotCom
Copy link
Member

Please run the pre-commit hooks for formatting, the lint action failed

discord/bot.py Outdated Show resolved Hide resolved
@Ratery Ratery changed the title Make get_desynced_commands work for slash command groups Make get_desynced_commands work with slash command groups Feb 16, 2022
@krittick
Copy link
Contributor

krittick commented Mar 7, 2022

Please resolve conflicts. Thanks!

@krittick krittick added this to the v2.0 milestone Mar 7, 2022
@BobDotCom BobDotCom self-assigned this Apr 4, 2022
…/pycord into Ratery-fix-get-desynced-commands

# Conflicts:
#	discord/bot.py
@BobDotCom BobDotCom changed the base branch from master to fix-registration April 4, 2022 15:53
@BobDotCom BobDotCom merged commit 493b4a5 into Pycord-Development:fix-registration Apr 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: medium Medium Priority status: in progress Work in Progess
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants