Skip to content

Conversation

@tristancamejo
Copy link
Member

Implements buttons into Disploy!

They function in the same way as commands, with the route system. I'm thinking of adding some sort of wildcard parsing instead of exact string matches for the custom_id match process so that you can do something like delete-:id and then get the id from the ButtonInteraction structure, something like ButtonInteraction#params#getString("id").

@tristancamejo tristancamejo added enhancement New feature or request cli apps:example packages:disploy structures Keeping our structures inline with the Discord API labels Nov 22, 2022
@tristancamejo tristancamejo added this to the v1.0.0 milestone Nov 22, 2022
@vercel
Copy link

vercel bot commented Nov 22, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Updated
disploy-docs ⬜️ Ignored (Inspect) Nov 23, 2022 at 10:53PM (UTC)

@tristancamejo
Copy link
Member Author

I will change the base branch to https://github.com/Disploy/disploy/tree/feat/message-components when it gets rebased, or should we merge straight to main? Keep in mind structures like action rows exist in that branch so the end-user doesn't have to use JSON through export type DiscordPayload<T> = T | unknown;.

@tristancamejo tristancamejo marked this pull request as ready for review November 23, 2022 08:43
Copy link
Member

@GodderE2D GodderE2D left a comment

Choose a reason for hiding this comment

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

Code changes outside of documentation and README files have not been reviewed. Please ask another person to review these changes.

tristancamejo and others added 3 commits November 24, 2022 09:34
Co-authored-by: GodderE2D <55918888+GodderE2D@users.noreply.github.com>
Co-authored-by: GodderE2D <55918888+GodderE2D@users.noreply.github.com>
Co-authored-by: GodderE2D <55918888+GodderE2D@users.noreply.github.com>
tristancamejo and others added 3 commits November 24, 2022 09:35
Co-authored-by: GodderE2D <55918888+GodderE2D@users.noreply.github.com>
Co-authored-by: GodderE2D <55918888+GodderE2D@users.noreply.github.com>
Copy link
Member

@GodderE2D GodderE2D left a comment

Choose a reason for hiding this comment

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

Non-documentation has not been reviewed

@tristancamejo tristancamejo merged commit d9d4577 into Disploy:main Nov 23, 2022
@tristancamejo tristancamejo deleted the feat/buttons branch November 23, 2022 23:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

apps:example cli enhancement New feature or request packages:disploy structures Keeping our structures inline with the Discord API

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants