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

chore(model): mark StickerPack.banner_asset_id Optional #1337

Merged
merged 2 commits into from
Dec 18, 2021
Merged

chore(model): mark StickerPack.banner_asset_id Optional #1337

merged 2 commits into from
Dec 18, 2021

Conversation

vilgotf
Copy link
Member

@vilgotf vilgotf commented Dec 17, 2021

@vilgotf vilgotf added c-model Affects the model crate d-api Change related to Discord's API. d-breaking Code breaks the API because of Discord m-breaking change Breaks the public API. t-chore labels Dec 17, 2021
@7596ff 7596ff merged commit c4a708d into twilight-rs:main Dec 18, 2021
@vilgotf vilgotf deleted the chore/model/sticker-banner-optional branch December 19, 2021 00:14
7596ff added a commit that referenced this pull request Dec 24, 2021
Changes

All `AuditLogChange::{new, old}` variants are now `Option`s ([#1324] -
[@7596ff]). This is to prevent small changes made by Discord from
causing deserialization errors. Additionally, adds some missing variants
from the latest docs.

`StickerPack::banner_asset_id` is now an `Option` ([#1337] -
[@vilgotf]).

Add `avatar`, `permissions`, and `pending` fields to `InteractionMember`
([#1339] - [@itohatweb]).

[#1324]: #1324
[#1337]: #1337
[#1339]: #1339

[@7596ff]: https://github.com/7596ff
[@itohatweb]: https://github.com/itohatweb
[@vilgotf]: https://github.com/vilgotf
@zeylahellyer zeylahellyer added this to the Discord API Catchup milestone Dec 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c-model Affects the model crate d-api Change related to Discord's API. d-breaking Code breaks the API because of Discord m-breaking change Breaks the public API.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants