Fix: sel plan page crud

[santipalenque]

https://app.clickup.com/t/86b87ekh9

Summary by CodeRabbit

• Refactor
  • Updated internal selection plan creation and management components

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

📝 Walkthrough

Walkthrough

Three components are refactored from class-based to functional components using React hooks. The selection-plan-layout routing is updated to use SelectionPlanIdLayout for the /new route instead of EditSelectionPlanPage.

Changes

Cohort / File(s)	Summary
Component modernization (class → functional) src/layouts/selection-plan-id-layout.js, src/pages/selection-plans/edit-selection-plan-page.js	Class components converted to functional components with hooks. SelectionPlanIdLayout uses useEffect with dependency on selectionPlanId. EditSelectionPlanPage converts instance-bound lifecycle methods to local arrow functions; event handlers (onDeleteExtraQuestion, onEditExtraQuestion, etc.) now defined inline and wired to SelectionPlanForm.
Routing refactor src/layouts/selection-plan-layout.js	Route for /new changed from EditSelectionPlanPage to SelectionPlanIdLayout, consolidating creation and editing flows into a single layout component.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~22 minutes

Poem

🐰 Class components hop away,
Hooks and functions lead the way,
SelectionPlans find a unified home,
The /new route will no longer roam!
Modernized code, clean and bright,
React functional—feels just right! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Title check	❓ Inconclusive	The title 'Fix: sel plan page crud' is vague and uses abbreviated terms (sel, crud) that don't clearly convey the main technical change—converting class components to functional components with hooks.	Consider a more descriptive title like 'Refactor: convert selection plan components to functional components with hooks' to clearly reflect the primary architectural change.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches

• 📝 Generate docstrings

---

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