papegaaij opened a new pull request, #1649: URL: https://github.com/apache/wicket/pull/1649
Fixes #1647. The WebSocket charts example parsed every pushed message with `jQuery.parseJSON`, which jQuery 4 removed. Master now defaults to jQuery 4, so every push threw `TypeError: jQuery.parseJSON is not a function` and the chart stopped updating. With the jQuery-free engine selected, jQuery isn't loaded at all. This PR uses `JSON.parse` instead, which works with either jQuery version and either engine. Checked in headless Chrome by loading the real `charts.js` with jQuery 4, the default engine and a stubbed Google Charts loader, then publishing a `/websocket/message`. Master's version throws; the fixed one draws the pushed record. I did not run the example itself in Jetty. `mvn clean verify -Pjs-test` is green. 🤖 Generated with [Claude Code](https://claude.com/claude-code) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
