-
-
Notifications
You must be signed in to change notification settings - Fork 130
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
fix(model): Deserialize unavailable guilds in GUILD_CREATE. #2330
Conversation
aac819b
to
48b1f66
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Correct me if I'm wrong, but I assume we don't test the cache's implementation of a guild going unavailable initially and then available again. Could that be added?
This changes GuildCreate from being a tuple struct to being a enum that can either be available or unavailable, and containing either a full Guild or a UnavailableGuild.
a5c2202
to
26e14fa
Compare
@Gelbpunkt Added a test now, which also made a small bug show up. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm thank u for the pr
This changes
GuildCreate
from being a tuple struct to being a enum that can either be available or unavailable, and containing either a fullGuild
or aUnavailableGuild
.