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

Tweak documentation for premium apps (aka. application subscriptions) #6452

Merged
merged 5 commits into from
Sep 28, 2023

Conversation

Lulalaby
Copy link
Contributor

@Lulalaby Lulalaby commented Sep 26, 2023

Follow-up for #6451

@Lulalaby Lulalaby force-pushed the imrpove-premium-docs branch 3 times, most recently from 46c45d9 to 1fd06ab Compare September 26, 2023 18:01
@Lulalaby Lulalaby changed the title improve app subscription docs Tweak documentation for premium apps (aka. application subscriptions) Sep 26, 2023
@DonovanDMC
Copy link
Contributor

| PREMIUM_REQUIRED\*\* | 10 | respond to an interaction with an upgrade button, only available for apps with monetization enabled |
\* Only valid for [component-based](#DOCS_INTERACTIONS_MESSAGE_COMPONENTS/) interactions
\*\* Not available for `MODAL_SUBMIT` and `PING` interactions.

This should be changed to say PREMIUM_REQURIED is not applicable to APPLICATION_COMMAND_AUTOCOMPLETE, not MODAL_SUBMIT. It works just fine on MODAL_SUBMIT interactions.

@Lulalaby
Copy link
Contributor Author

Will do, tho that throws a question in my head.
Why can't that be send for autocomplete.
That's kinds weird.

Co-Authored-By: Donovan Daniels <hewwo@yiff.rocks>
@sebm253 sebm253 mentioned this pull request Sep 27, 2023
2 tasks
@Misha-133
Copy link
Contributor

Should this also do something with this
Feels kinda wrong lol, example has undocumented fields
image

Lulalaby and others added 2 commits September 28, 2023 19:41
Co-authored-by: Colin Loretz <colin.loretz@discordapp.com>
@colinloretz colinloretz merged commit 70b03d1 into discord:main Sep 28, 2023
3 checks passed
@Lulalaby Lulalaby deleted the imrpove-premium-docs branch September 28, 2023 18:01
mccoderpy added a commit to mccoderpy/discord.py-message-components that referenced this pull request Nov 4, 2023
shaydewael pushed a commit to Jupith/discord-api-docs that referenced this pull request May 14, 2024
…discord#6452)

* improve app subscription docs

* mention where premium required can't be used

Co-Authored-By: Donovan Daniels <hewwo@yiff.rocks>

* Update docs/monetization/SKUs.md

Co-authored-by: Colin Loretz <colin.loretz@discordapp.com>

* fix table

* restore consumed (for now)

---------

Co-authored-by: Donovan Daniels <hewwo@yiff.rocks>
Co-authored-by: Colin Loretz <colin.loretz@discordapp.com>
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 this pull request may close these issues.

None yet

4 participants