Skip to content

Create FRQ editor page skeleton - #325

Merged
saa938 merged 2 commits into
AP-Students:mainfrom
TechnoSamurai02:issue-305-frq-editor-skeleton
Jul 22, 2026
Merged

Create FRQ editor page skeleton#325
saa938 merged 2 commits into
AP-Students:mainfrom
TechnoSamurai02:issue-305-frq-editor-skeleton

Conversation

@TechnoSamurai02

@TechnoSamurai02 TechnoSamurai02 commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Adds the admin FRQ editor page and extracts the subject, unit, and FRQ ID from the URL.
  • Fetches the FRQ from the required nested Firestore path.
  • Adds a plain-text renderer that displays different messages when the FRQ is found or missing.

Testing

  • npm run lint — passed with existing unrelated warnings.
  • npm run build — passed.
  • Tested both existing and missing FRQ documents using the local Firebase emulator.

Closes #305

@TechnoSamurai02
TechnoSamurai02 marked this pull request as ready for review July 20, 2026 16:44
@Famousmaster206
Famousmaster206 requested a review from Copilot July 20, 2026 18:43
Copilot AI reviewed Jul 20, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@Famousmaster206
Famousmaster206 requested a review from Copilot July 20, 2026 18:59
Copilot AI reviewed Jul 20, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

Comment thread src/app/admin/subject/[slug]/[unit]/frq/[id]/page.tsx Outdated
saa938
saa938 previously approved these changes Jul 21, 2026
View reviewed changes

@saa938 saa938 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@saa938
Potential fix for pull request finding
7a31ba1 
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
saa938
saa938 approved these changes Jul 22, 2026
View reviewed changes

@saa938 saa938 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@saa938
saa938 merged commit 7eba68f into AP-Students:main Jul 22, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
@saa938 saa938 saa938 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

INTERN - FRQ Editor Page Skeleton

3 participants

@TechnoSamurai02 @saa938