Skip to content

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
@Ratery Ratery requested a review from BobDotCom February 12, 2022 15:04
@BobDotCom
Copy link
Contributor

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

@Ratery Ratery force-pushed the fix-get-desynced-commands branch from 58a3df7 to 9f4bb9e Compare February 12, 2022 18:07
@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