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
Uh oh!
There was an error while loading. Please reload this page.
-
Prerequisite for #1070
Summary
Migrate existing instructors with their
instructor_ids 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_viewsand 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