Replies: 1 comment
-
Syncing instructors may be out of scope for this project. It would be more than what is needed for the course builder |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Prerequisite for #1070
Summary
Migrate existing instructors with their
instructor_id
s to Santiy and ensure that future updates to instructors are synced.Problem
Instructors need to be stored in both the [[egghead-rails]] database and [[sanity]]. There are different concepts in each of those systems though.
In egghead-rails, instructors are an accounting concept. They are tied to
lesson_views
and rolled up into royalty revenue share runs each month.In Sanity, instructors are a presentational / content concept. They are tied to lessons, courses, articles, and other content as the collaborator.
Desired State
Each instructor should be present in Sanity with a filled instructor_id field.
Solution
Update the Sanity schema and create a dropdown for Collaborator types. Currently it's a plain text field.
Migrate all the existing instructors in Rails into Sanity.
We also need to ensure that when a new instructor is created, or an existing instructor is updated on egghead-rails that they are kept in sync with Sanity.
Scope and Boundaries
Since instructors are already first created in egghead-rails as a key piece of the accounting story, it makes sense for them to continue to be created there (even if we the admin UI for doing it lives at a different domain).
No changes to Instructors on the egghead-rails side. In the future we can probably remove columns with presentation data like avatar, bio, socials, etc.
Beta Was this translation helpful? Give feedback.
All reactions