discord.py icon indicating copy to clipboard operation
discord.py copied to clipboard

Support commands.parameter for hybrid app commands

Open Soheab opened this issue 6 months ago • 0 comments

Summary

Bit of a misleiding title but what this PR basically does is copy over the displayed_name and description from commands.Parameter if commands.(param)eter is used on a hybrid command param because most people expect this to happen anyways and I think it makes sense.

I don't know if any docs changes are needed here.

Breaking changes

This could be a breaking change for people that don't expect this to happen and may want a different description for the prefix part of the hybrid command or don't want displayed_name to rename it... should there maybe be a toggle for this?

Checklist

  • [x] If code changes were made then they have been tested.
    • [ ] I have updated the documentation to reflect the changes.
  • [ ] 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, ...)

Soheab avatar Jul 28 '24 13:07 Soheab