Reimplement the "In Progress" section on the egghead homepage
MaggieAppleton opened this issue · 2 comments
Back in April of 2021 we implemented an "in progress" component that helped learners pick up where they left off.
It showed logged in users the last 3 courses they watched at the top of their home page.
Original pitch: #551
When we deployed the component, we found a number of users were experiencing bugs and quickly rolled it back: #558
The issue related to the SeriesProgress value in our GraphQL requests conflicting with progress data coming in from client websites (epic react, testing js): https://github.com/eggheadio/egghead-rails/issues/4476
If the user has watched any lessons from a WBaaS site (pure react, testing js, epic react etc) this query will break because we are adding the x-site-client header to the graphql request.
We need to tenant the SeriesProgress model to resolve this. There are 2_551_357 SeriesProgress objects so we have to be careful with this data migration.
@theianjones began working on this with help from @jbranchaud: https://github.com/eggheadio/egghead-rails/pull/4487
It's unclear whether the issue with SeriesProgress has been resolved.
Open question: have we fixed the conflicting data between our client websites and egghead? Is it safe to try and reimplement this component?
It looks like for the vast majority of Series Progress records, we now have the relevant site recorded. If this wasn't working before because we didn't know what records belong with what Site Tenant, then it should now have the data it needs.
Is it actual still? cc @laurosilvacom