Feat/add fcc proper user #571
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Removed isAdminApproved field that was causing schema drift - Field was in migration but not in schema.prisma - Resolves database sync issues
- Added missing migration for refresh_token_expires_in field - Field exists in schema.prisma but was missing from database - Resolves Account table schema drift
- Added fccProperUserId as optional unique field - Prepares for FCC Proper API integration - Enables linking classroom users with FCC proper accounts
- Removed old migration with isAdminApproved drift issue - Created fresh migration from current schema.prisma - Database schema now matches schema definition exactly - No isAdminApproved field (was causing drift) - Includes all current schema fields (fccProperUserId, refresh_token_expires_in)
utsab
approved these changes
Dec 9, 2025
2 tasks
CarlyAThomas
added a commit
to CarlyAThomas/classroom
that referenced
this pull request
Dec 23, 2025
* fix: remove isAdminApproved from User table migration - Removed isAdminApproved field that was causing schema drift - Field was in migration but not in schema.prisma - Resolves database sync issues * fix: add refresh_token_expires_in column to Account table - Added missing migration for refresh_token_expires_in field - Field exists in schema.prisma but was missing from database - Resolves Account table schema drift * feat: add fccProperUserId field to User model - Added fccProperUserId as optional unique field - Prepares for FCC Proper API integration - Enables linking classroom users with FCC proper accounts * fix: replace old migration with clean initial setup - Removed old migration with isAdminApproved drift issue - Created fresh migration from current schema.prisma - Database schema now matches schema definition exactly - No isAdminApproved field (was causing drift) - Includes all current schema fields (fccProperUserId, refresh_token_expires_in)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Please note:
⚠️ Depends on: PR #569 and PR #570 - Please review and merge those PRs first.
Changes made:
fccProperUserIdas optional unique field to User model inschema.prisma20251207074222_add_fcc_proper_user_idChecklist:
Update index.md)This PR is not associated with an issue.
Overview
Adds
fccProperUserIdfield to User model to enable integration with the FCC Proper API.Purpose
This field will be used to:
Technical Details
fccProperUserId String? @unique