Skip to content

feat: implement admin-specific functionalities - #88

Merged
mikelReinanArcardo merged 38 commits into
up-csi:mainfrom
rue-22:main
Jul 16, 2026
Merged

feat: implement admin-specific functionalities#88
mikelReinanArcardo merged 38 commits into
up-csi:mainfrom
rue-22:main

Conversation

@rue-22

@rue-22 rue-22 commented May 12, 2026

Copy link
Copy Markdown
Collaborator

This PR is the whole implementation of the admin-specific functionalities for the UP CSI App. It also includes some code cleanups of the repo.

At a very high level, the features implemented here are based on the use cases documentation in this Google Sheets.

tm-cj-salces and others added 30 commits March 24, 2026 11:13
Adds a new admin-queries function that returns all applicants
(including those with no quiz activity), each tagged with a
Not Started / In Progress / Completed status derived from
constiquiz-submissions and constiquiz-answers, plus the quiz's
max possible score (sum of question point_values). Used by the
upcoming /admin/constiquiz respondent list page.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Encapsulates the list-page layout (heading + search + filter +
sort + paginated table) and the four state variables those
controls share. Pages just supply the title, data, columns,
filter key, row-click handler, and a cell snippet for custom
rendering. Will be reused by the constiquiz and sigsheet
admin list pages.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Implements P02-001. The /admin/constiquiz route lists every
applicant with their status badge, current score over max
possible score, and a "View response" link to the (future)
P02-002 detail page. Loader calls fetchQuizRespondents and
the page composes AdminListView with custom cell rendering
for the status badge, score format, and link column.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Removed accidental `sign`/`string` imports from admin-queries.ts that
  broke `pnpm build` (effect/FastCheck is not a dependency).
- Renamed route `/admin/sigsheet/[id]` to `[userId]` to match the link
  emitted by the P03-001 list page, and updated the loader to use
  `params.userId`.
- Removed a redundant null-coalesce/cast on `sigsheetData` in
  `fetchSigsheetDetail`.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@rue-22
rue-22 requested a review from ehrelevant May 12, 2026 08:17
@mikelReinanArcardo
mikelReinanArcardo self-requested a review July 16, 2026 13:11
@mikelReinanArcardo
mikelReinanArcardo merged commit e81e5aa into up-csi:main Jul 16, 2026
1 check passed
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.

5 participants