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 saveMinecraft into the server status #127
Add saveMinecraft into the server status #127
Conversation
This massively speeds up sync times for me, since Loom no longer has to remap 45+ Fabric API modules. Additionally, this allows users to install No Chat Reports without Fabric API, as the required modules are already included.
Added Finnish translation. // Lisätty suomenkielinen käännös.
…ation Spanish translation
private void onSerialize(ServerStatus serverStatus, Type type, JsonSerializationContext context, | ||
CallbackInfoReturnable<JsonElement> info) { | ||
if (!NoReportsConfig.includeQueryData()) return; | ||
((JsonObject)info.getReturnValue()).addProperty("noChatReporting", NoReportsConfig.convertToGameMessage()); |
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.
Why have you decided to tie noChatReporting
value to message conversion? NCR removes signatures both ways, conversion is just an optional alternative strategy for doing that.
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.
Also there's still that bunch of conflicts to resolve. I hope it's not too much to ask from you to take care of them.
…dMenu-Cloth Adds ModMenu support for 1.19.2
plus added a newline to option.NoChatReports.whitelistedServers.tooltip
Closed in favor of more clean and up-to-date implementation in #169. |
You should display a small icon next to servers that support this. So show that no chat report is used on the server, should be easy to add the icon with what ive done