Skip to content

Conversation

@TheSKBroook
Copy link
Contributor

@TheSKBroook TheSKBroook commented Sep 11, 2025

Description

Semester Creation Flow

The semester creation flow is managed using useReducer (based on the patterns from bookFlow.tsx and bookFlowReducer.tsx) instead of useState to handle multiple states more effectively.

Flow States

  1. Semester Name
  2. Semester Date
  3. Semester Break Date
  4. Success Message

Notes

  • Only the logic flow has been implemented.
  • No backend integration is wired up yet.
  • The final frontend UI is not included at this stage -- PlaceHolder to display the flow for now

Demo

demo.for.712.mp4

Fixes #712

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

  • Manual testing (requires screenshots or videos)
  • Integration tests written (requires checks to pass)

Checklist before requesting a review

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added thorough tests that prove my fix is effective and that my feature works
  • I've requested a review from another user

@github-actions
Copy link
Contributor

github-actions bot commented Sep 11, 2025

Preview Deployment Status

✅ Storybook Preview: https://ad32151e.uabc-storybook.pages.dev

✅ Frontend Preview: https://02723e77.uabc.pages.dev

@github-actions
Copy link
Contributor

Coverage Report

Status Category Percentage Covered / Total
🟢 Lines 84.49% (🎯 70%)
⬇️ -1.34%
2223 / 2631
🟢 Statements 84.3% (🎯 70%)
⬇️ -1.49%
2282 / 2707
🔴 Functions 79.62% (🎯 80%)
⬇️ -2.27%
547 / 687
🟢 Branches 78.18% (🎯 60%)
⬇️ -1.57%
1200 / 1535
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
apps/frontend/src/components/client/admin/AdminSection.tsx 0%
🟰 ±0%
100%
🟰 ±0%
0%
🟰 ±0%
0%
🟰 ±0%
9-36
apps/frontend/src/components/client/admin/AdminSemesterFlow.tsx 0% 0% 0% 0% 7-75
apps/frontend/src/components/client/admin/AdminSemesterSection.tsx 0% 0% 0% 0% 7-16
apps/frontend/src/components/client/admin/adminSemesterFlowReducer.ts 0% 0% 0% 0% 31-84
Generated in workflow #3353 for commit 02f061f by the Vitest Coverage Report Action

@TheSKBroook
Copy link
Contributor Author

TheSKBroook commented Sep 11, 2025

@jeffplays2005 I am a little unsure on the scope of this ticket.
Do I need to link it to the bankend when the semester successfully created or will it be in another ticket?
And do I need to check the date of the semester, break ( e.g break after semester )

Also the ui components as well?

@TheSKBroook TheSKBroook marked this pull request as draft September 11, 2025 03:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FRONTEND] Extend Admin semesters view to create a Semester

2 participants