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

fix: correct keys on slot packets #1954

Merged
merged 1 commit into from
Oct 15, 2023

Conversation

lodicolo
Copy link
Member

Fixes #1953 by correcting the [Key(#)] attributes on the different slot packets

For some reason the base slot packet had [Key(3)] on a bool property with only an expression-body getter (so quite literally cannot be set even with reflection hacks), and the deposit bank slot was [Key(0)].

It appears that there isn't actually a [Key(0)] on anything? Which is weird, but whatever. The problem is that the weird key situation was causing deserialization to totally break, and I am guessing since our .NET 7 upgrade MessagePack actually expects the code to be correct instead of correcting for developer mistakes.

@lodicolo lodicolo merged commit 361d63d into AscensionGameDev:main Oct 15, 2023
1 check passed
@lodicolo lodicolo deleted the fix-1953-slot-packet-keys branch October 15, 2023 18:34
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.

Bug Bank/Withdraw does not work
1 participant