Implement modals

[thesadru]

References

• official notion post
• API documentation
• current partial implementation in disnake
• discord-payloads
• Disnake Beta Tests
• @Victorsitou helping with the details

Checklist

ETA: 2 months ago

• [x] Make all pipelines pass
• [x] modal interaction response (sent to user)
  • [x] builder
  • [x] rest method
• [x] modal interaction event (gotten from user)
  • [x] all attributes
  • [x] all helper methods
  • [x] respond with rest
  • [x] respond with restbot
• [x] text input
  • [x] interaction model
  • [x] response builder
    • [x] builder classes
    • [x] helper methods
• [x] factory
  • [x] modal
  • [x] text input
• [x] tests
  • [x] modal interaction response
    • [x] rest
    • [x] helper methods
  • [x] modal interaction response builder
    • [x] rest
    • [x] helper methods
  • [x] modal interaction event
    • [x] entity factory
    • [x] methods
  • [x] text input builder
• [ ] suggestions
  • [ ] helper method to add a single text input wrapped in an action row

[hypergonial]

Been testing it for about a week now, seems ready for final review.

[thomm-o]

An issue with adding create_modal_response to MessageResponseMixin - you cannot respond to a modal with another modal so this gives incorrect suggestions for IDE autocomplete.

[tameTNT]

Is it possible to get multiline text input via a slash command in hikari or are these modals the only 'new and approved' (i.e. not traditional prefix command-based) option available?

[GoogolGenius]

Is it possible to get multiline text input via a slash command in hikari or are these modals the only 'new and approved' (i.e. not traditional prefix command-based) option available?

This is a discord limitation

[tameTNT]

Sounds about right... 🙄

Thanks for the swift reply anyway - I'll just pray for this PR to be merged soon then 🙏

[hypergonial]

ModalInteraction probably needs app_permissions field added.

[thesadru]

tbh ://////

[FasterSpeeding]

If ya could get some of ye last review thingies fixed i'da think this would be bout ready to merge