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

ApplicationCommand missing fields #1202

Closed
JABirchall opened this issue Jun 20, 2022 · 4 comments · Fixed by #1207
Closed

ApplicationCommand missing fields #1202

JABirchall opened this issue Jun 20, 2022 · 4 comments · Fixed by #1207

Comments

@JABirchall
Copy link

JABirchall commented Jun 20, 2022

DiscordGo v0.25 is missing fields for ApplicationCommands documented here: https://discord.com/developers/docs/interactions/application-commands#application-command-object

  • default_member_permissions - Implemented on master
  • dm_permission - Implemented on master
  • guild_id

default_permissions will soon be removed infavor of default_member_permissions.
Registering commands via the guilds endpoint will soon be depricated infavor of restricting a command to a guild on the ApplicationCommand object

@SiirRandall
Copy link

SiirRandall commented Jul 1, 2022

Having the same issues with the examples.
ApplicationCommandPermissionTypeChannel not declared by package discordgo compiler UndeclaredImportedName

@FedorLap2006
Copy link
Collaborator

FedorLap2006 commented Jul 2, 2022

Having the same issues with the examples. ApplicationCommandPermissionTypeChannel not declared by package discordgo compiler UndeclaredImportedName

Permissions aren't released yet. If you want to use them, you must have the master branch:

$ go get github.com/bwmarrin/discordgo@master

@FedorLap2006
Copy link
Collaborator

FedorLap2006 commented Jul 2, 2022

DiscordGo v0.25 is missing fields for ApplicationCommands documented here: https://discord.com/developers/docs/interactions/application-commands#application-command-object

  • default_member_permissions - Implemented on master
  • dm_permission - Implemented on master
  • guild_id

default_permissions will soon be removed infavor of default_member_permissions. Registering commands via the guilds endpoint will soon be depricated infavor of restricting a command to a guild on the ApplicationCommand object

Sorry for a late response, can you link the documentation/changelog on where the "registering commands via ..." behaviour is described? Can't seem to find it anywhere.

@JABirchall
Copy link
Author

JABirchall commented Jul 2, 2022

It has yet to be announced. Its been discussed to remove duplicate endpoints.

Where instead of doing an if check for guild_id to check if its a guild command or global command. It will only need to be set on the command object.

@FedorLap2006 FedorLap2006 linked a pull request Jul 2, 2022 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants