Replace term confusion pairs with review events#28
Conversation
This change replaces the old `term_confusion_pairs` table and related GraphQL structures with a new `review_events` system. Review events are now recorded for every question in a practice test, capturing detailed information about correctness, answer type, and specific terms confused. Key changes: - New `review_events` table and `review_activity_type_enum`. - Removed `recordConfusedTerms` mutation and `TermConfusionPair` type. - Updated `RecordPracticeTest` to populate `review_events`. - Updated `UpdatePracticeTestQuestion` to maintain consistency with `review_events`. - Cleaned up related resolvers, loaders, and models.
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
ehanahamed
force-pushed
the
replace-confusion-pairs-with-review-events-14945051930215221223
branch
from
daa9f9b to
c44bb1e
Compare
ehanahamed
deleted the
replace-confusion-pairs-with-review-events-14945051930215221223
branch
1 participant
Replaced the deprecated term confusion pairs system with a new review events system that logs detailed results for each practice test question. This includes database migrations, GraphQL schema updates, and resolver logic for MCQs, TFQs, and FRQs.
PR created automatically by Jules for task 14945051930215221223 started by @ehanahamed