Skip to content

Fix playerlist serialisation + error logging - #115

Open
Zoriot wants to merge 2 commits into
issue-62-nav-submenu-commandsfrom
fix/nwapi
Open

Zoriot wants to merge 2 commits into
issue-62-nav-submenu-commandsfrom
fix/nwapi

Conversation

@Zoriot

@Zoriot Zoriot commented Oct 11, 2026

Copy link
Copy Markdown
Collaborator

Due to parsing error's the request contained the raw array.toString which lead to error on the nwapi. Now it correctly serialized it as json.

Unchecked calls were also removed.

@Zoriot Zoriot added this to the Release v0.4.0 milestone Oct 11, 2026
@Zoriot
Zoriot requested a review from a team October 11, 2026 12:43
@Zoriot Zoriot self-assigned this Oct 11, 2026
@Zoriot Zoriot added the 🚨 high priority This currently has high priority. label Oct 11, 2026
@Zoriot
Zoriot added this pull request to stack #113 October 11, 2026 12:43
@github-actions

github-actions Bot commented Oct 11, 2026 •

Copy link
Copy Markdown

Qodana for JVM

It seems all right 👌

No new problems were found according to the checks applied

💡 Qodana analysis was run in the pull request mode: only the changed files were checked
☁️ View the detailed Qodana report

Contact Qodana team

Contact us at qodana-support@jetbrains.com

@Zoriot
Zoriot requested a review from Jasupa October 11, 2026 20:20
@Zoriot
Zoriot force-pushed the fix/nwapi branch 3 times, most recently from 9426f32 to d7286b5 Compare October 11, 2026 20:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🚨 high priority This currently has high priority.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant