-
-
Notifications
You must be signed in to change notification settings - Fork 211
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
Add world save mode for ServerWorld#save() to support flush #3688
Conversation
7e68793
to
d26c21f
Compare
e012fbc
to
806bf13
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.
There are some changes in behaviour with serialization behaviour and events that seem unrelated to this PR.
...ins/java/org/spongepowered/common/mixin/api/minecraft/server/level/ServerLevelMixin_API.java
Outdated
Show resolved
Hide resolved
src/mixins/java/org/spongepowered/common/mixin/core/server/level/ServerLevelMixin.java
Outdated
Show resolved
Hide resolved
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.
I have made the fixes to the SerializationBehavior logic in 3a371bb that were discussed on Discord.
We can take a look at how the events should be fired on api-10 but for api-8 it would be a breaking change.
28c8124
to
40751d6
Compare
40751d6
to
2333bc2
Compare
2333bc2
to
91657c9
Compare
e866f33
to
13c9946
Compare
5825843
to
1c4ba14
Compare
SpongeAPI | Sponge