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

Role subscriptions #743

Merged
merged 7 commits into from
Mar 4, 2023
Merged

Conversation

NoComment1105
Copy link
Contributor

discord/discord-api-docs@e003051

The JSON error codes are in #742

@HopeBaron
Copy link
Member

common.api is conflicting so it would be better to merge this after we add the other PR

@NoComment1105
Copy link
Contributor Author

I can run a quick apiDump if necessary

@HopeBaron HopeBaron merged commit 41c9560 into kordlib:0.8.x Mar 4, 2023
lukellmann added a commit to lukellmann/kord that referenced this pull request Mar 4, 2023
* rename GuildFeature.CreatorMonetizableProvision to
  GuildFeature.CreatorMonetizableProvisional

* deprecate GuildFeature.MonetizationEnabled, see
  discord/discord-api-docs#5724 (comment)

* add @SerialName for DiscordRoleTags.botId

* add RoleTags.isLinkedRole

* fix DiscordMessage.roleSubscriptionData nullability

* fix KDocs

see discord/discord-api-docs#5724 and
discord/discord-api-docs#5828
lukellmann added a commit that referenced this pull request Mar 5, 2023
* rename GuildFeature.CreatorMonetizableProvision to
  GuildFeature.CreatorMonetizableProvisional

* deprecate GuildFeature.MonetizationEnabled, see
  discord/discord-api-docs#5724 (comment)

* add @SerialName for DiscordRoleTags.botId

* add RoleTags.isLinkedRole

* fix DiscordMessage.roleSubscriptionData nullability

* fix KDocs

see discord/discord-api-docs#5724 and
discord/discord-api-docs#5828

Co-authored-by: NoComment <67918617+NoComment1105@users.noreply.github.com>
@NoComment1105 NoComment1105 deleted the role-subscriptions branch March 25, 2023 14:58
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.

2 participants