Issue: Duplicate Routes in App.jsx Causing Redundant Routing Definitions

[Shantanu675]

🐞 Duplicate Routes in App.jsx Causing Redundant Routing Definitions

Description

There are duplicate route definitions in App.jsx, specifically related to the Coach routes. These duplicates can lead to routing ambiguity, unexpected behavior, and reduced maintainability of the routing configuration.

Affected File

src/App.jsx

Problematic Code

<Route path='/coach' element={<CoachPage />} />
<Route path='coach/strengthen-argument' element={<StrengthenArgument />} />
<Route path='/coach' element={<CoachPage />} />

<Route path='coach/strengthen-argument' element={<StrengthenArgument />} />

Why This Is an Issue

• Duplicate route declarations violate the DRY (Don’t Repeat Yourself) principle
• Can cause unexpected routing resolution behavior
• Makes future refactoring more error-prone
• Reduces overall readability and maintainability

Expected Behavior

Each route should be defined only once, with a clear and consistent routing structure.

Suggested Fix

Remove the duplicate route entries and keep a single, well-defined Coach route configuration.

<Route path='coach' element={<CoachPage />} />
<Route path='coach/strengthen-argument' element={<StrengthenArgument />} />

Additional Notes

This change improves code clarity and prevents potential routing conflicts without altering existing functionality.

Contribution

I would be happy to work on this issue and submit a pull request to address it.