Skip to content

Revert "Frq feature " - #377

Merged
saa938 merged 1 commit into
mainfrom
revert-376-frq
Aug 20, 2026
Merged

Revert "Frq feature "#377
saa938 merged 1 commit into
mainfrom
revert-376-frq

Conversation

@saa938

@saa938 saa938 commented Aug 20, 2026

Copy link
Copy Markdown
Collaborator

Reverts #376

Copilot AI lite review requested due to automatic review settings August 20, 2026 01:59

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.

@saa938
saa938 merged commit 6b3128f into main Aug 20, 2026
1 check passed
saa938 added a commit that referenced this pull request Aug 21, 2026
main carries commit ff5fcdc, the blanket revert of PR #376 ("Frq feature").
The frq branch was then rebuilt from the pre-revert merge base c528939 and
the bugs that caused the revert were fixed in PRs #379-#382. Because the
rebuild produced new commit SHAs, git cannot relate the revert to the work
that supersedes it, so every FRQ file collides: 18 conflicts, plus 10 files
that main deleted and frq never touched, which git resolves silently as
deletions.

main has no independent changes since the merge base. Its only commits are
the #376 merge, the revert, and the #377 merge, so every conflicting hunk on
main's side is reverted pre-FRQ code that the fix PRs replace. The merge is
therefore resolved to the frq tree in full, which also restores the 10 files
that would otherwise have been dropped: firebase.json, firestore.indexes.json,
frqRefs.ts, responseEditor.tsx, FRQDropdown.tsx, FRQFooter.tsx, unitFrqs.tsx,
and feedback/header.tsx, feedback/leftSide.tsx, feedback/types.ts.

Verified: next build succeeds and emits all five FRQ routes (authoring,
taking, /frq-grading, /frq-grading/[id], /frq-feedback/[id]).
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.

2 participants