-
Notifications
You must be signed in to change notification settings - Fork 172
feat: add programs dashboard #751
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #751 +/- ##
==========================================
- Coverage 97.54% 97.18% -0.37%
==========================================
Files 148 155 +7
Lines 1302 1419 +117
Branches 225 247 +22
==========================================
+ Hits 1270 1379 +109
- Misses 31 39 +8
Partials 1 1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
@openedx/openedx-product-managers this is ready for review |
|
@MaxFrank13, can you make codecov any happier? |
deborahgu
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm approving based on our prior reviews of this feature branch, but obviously wait for product manager review before merging.
|
Note for @openedx/openedx-product-managers -- this is list page only. The details page is still legacy. |
Related Github Issue in platform roadmap
[Proposal] Legacy Program Dashboard conversion
This PR adds the programs dashboard in accordance with the above proposal. This is a conversion of the legacy programs dashboard that lives in edx-platform. This PR converts the legacy frontend into a React based frontend that lives under its own route. The route is conditionally rendered based on a new
ENABLE_PROGRAM_DASHBOARDenvironment variable, not to be confused with theENABLE_PROGRAMSvariable, which only handles the rendering of the "Programs" tab. This is done so that operators can choose to either use the legacy frontend or the new React-based frontend.In order to create a new route in this MFE, the
App.jsxfile had to be refactored. TheLearnerDashboardHeaderandFooterSlotwere moved out ofApp.jsxand intoindex.jsx. This aligns with the way other MFEs are setup. Theh1tag for the app was also moved to theLearnerDashboardHeaderso that it would appear on all routes. The Header logic has also been refactored so that the correct tab is highlighted based on the pathname of the page.All other changes are related to the Program Dashboard itself. The dashboard uses the
/api/dashboard/v0/programs/endpoint to retrieve enrollment data.Directory structure
Programs Dashboard
Has enrollments
Does not have enrollments
Error retrieving enrollment data