fix: redirect index to new play page

[hazikchaudhry]

Issue

The website was directing to the old play page

Solution

the default url now re-directs to the new play page, using the same naming routes as the rest of the routes

/play -> play page
/create -> create scenario
/dashboard -> dashboard access

Risk

I havent fully testing it, so it should be fine but it depends. But it should work, unless there is a freaky bug

Checklist

• Acceptance criteria met
• Wiki documentation is written and up to date
• Unit tests written and passing
• Integration tests written and passing
• Continuous integration build passing

Summary by CodeRabbit

• Navigation Updates
  • Modified application routing structure; the home page now redirects to the play section instead of displaying the scenario selection interface.

[linear]

VPS-72

[harbassan]

just delete the old page 😭 get that piece of shit out of our codebase pls

[harbassan]

@hazikchaudhry read my comment

[coderabbitai]

📝 Walkthrough

Walkthrough

App.jsx router configuration is updated: Redirect is added to the react-router-dom import, ScenarioSelectionPage is removed as an import, the protected root route (/) now redirects to /play instead of rendering ScenarioSelectionPage, and the protected /play-page route rendering PlayPage is removed entirely.

Router Navigation Updates

Layer / File(s)	Summary
Redirect import, route change, and play-page removal frontend/src/App.jsx	Redirect is added to react-router-dom imports; ScenarioSelectionPage import is removed; the / protected route is replaced with a <Redirect to="/play" />; the /play-page protected route rendering PlayPage is deleted.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~5 minutes

Poem

🐇 Hippity hop, the routes have changed,
No more selection page to be arranged!
The root now leaps to /play with glee,
And /play-page has been set free.
A redirect's hop — short, swift, and neat! 🌿

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description check	✅ Passed	The description addresses all required template sections with sufficient detail about the issue, solution, and risks, though the checklist items remain incomplete.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Title check	✅ Passed	The title accurately describes the main change: redirecting the index route to the new play page, which aligns with the core modification in App.jsx.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch vps-72-get-rid-of-the-old-play-page

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}