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

Application fields changes #6484

Closed
wants to merge 1 commit into from
Closed

Conversation

Puncher1
Copy link
Contributor

This PR adds following missing application fields:

  • discoverability_state
  • discovery_eligibility_flags
  • explicit_content_filter
  • hook
  • interactions_event_types
  • interactions_version
  • is_monetized
  • monetization_eligibility_flags
  • monetization_state
  • rpc_application_state
  • store_application_state
  • verification_state

and changes following fields:

  • role_connections_verification_url type from string to ?string

Copy link
Contributor

@JustinBeckwith JustinBeckwith left a comment

Choose a reason for hiding this comment

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

Thanks for the PR! While a lot of these fields may be included in the response, not all fields that come back in a response are part of the documented, supported API. For that - you really want to be looking at what's available in the Open API spec: https://github.com/discord/discord-api-spec/blob/main/specs/openapi.json#L26277

The only field that should be included is explicit_content_filter.

@JustinBeckwith JustinBeckwith added the waiting for response Discord is waiting for a response and will re-triage the issue at a later point label Sep 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
waiting for response Discord is waiting for a response and will re-triage the issue at a later point
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants