Conversation
Co-authored-by: andrew <andrew@anysphere.co>
|
Your pull request is now ready for review with Assert. Stop waiting for your code to break. Ship with confidence using Assert. |
|
Cursor Agent can help with this pull request. Just |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
| setLoadParkId(null); | ||
| setPendingRoomCode(null); | ||
| router.push('/'); | ||
| }, [multiplayer, router, setShowGame, setStartFresh, setLoadParkId, setPendingRoomCode]); |
There was a problem hiding this comment.
Duplicated cleanup logic between two handler functions
Medium Severity · Code Quality
handleBackToIsoCity duplicates 5 lines of cleanup logic from handleExitGame (lines 306-310). Both functions call multiplayer?.leaveRoom(), setShowGame(false), setStartFresh(false), setLoadParkId(null), and setPendingRoomCode(null). Extract this common cleanup logic into a shared helper function.
Localization Complete! ✅This PR has been successfully localized! 📋 SummaryLocadex has analyzed your changes and automatically localized your translatable content. Files internationalized (1)
Translation files updated (8)
Configuration files updated (2)
🔗 What happened?
Localization completed by Locadex in 342 seconds. |


This pull request contains changes generated by a Cursor Cloud Agent
Note
Low Risk
Small, UI-only navigation change; minimal risk beyond potential routing/regression in back-navigation behavior.
Overview
Updates the coaster landing page’s “Back to IsoCity” control to use Next.js client-side navigation (
useRouter().push('/')) instead of a plain link, and ensures any multiplayer room is left and local coaster UI state is reset before navigating away.Written by Cursor Bugbot for commit a0da117. This will update automatically on new commits. Configure here.