Create assigned_homeroom_id key on Educator table, copy to bootstrap, and update importers #2123
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.
Who is this PR for?
educators, developers
What problem does this PR fix?
Part of #1880 (comment).
Multiple educators can be assigned to the same homeroom, but Insights doesn't allow this now. This results in the importer process thrashing each day over who is assigned to the homeroom, and effectively alternating who will be authorized to view those students. This isn't user-impacting right now, and it came up from validating the Bedford imports.
The overall idea is to migrate the key to the Educator model, and do this in pieces:
Because these are related, once we start we should finish them all to avoid allowing drift during the migration.
What does this PR do?
This PR aims to do the first two items.
Checklists