Skip to content

feat: [FC-86] Added course overview and updated sidebar - #21

Closed
peterkulko wants to merge 1 commit into
Peter_Kulko/create-course-about-sidebarfrom
Peter_Kulko/course-about-overview
Closed

peterkulko wants to merge 1 commit into
Peter_Kulko/create-course-about-sidebarfrom
Peter_Kulko/course-about-overview

Conversation

@peterkulko

@peterkulko peterkulko commented Jul 31, 2025

Copy link
Copy Markdown

Warning

  • Merging all dependencies

Description

This PR adds an overview block and additional sidebar elements to the Course About page.

Useful information

Figma design

Initial setup

  1. Clone and install the tutor-mfe plugin from the draft PR that adds support for Catalog MFE.
  2. Go to http://apps.local.openedx.io:1998/catalog/

How Has This Been Tested?

  1. Open catalog/courses/<course_id>/about Course About page
  2. Ensure the content of the sidebar matches the legacy version
  3. Try changing course overview through the Schedule and details page

@peterkulko peterkulko self-assigned this Jul 31, 2025
@peterkulko peterkulko changed the title feat: added course overview and updated sidebar feat: [FC-86] Added course overview and updated sidebar Jul 31, 2025
@codecov

codecov Bot commented Jul 31, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 86.95652% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 94.53%. Comparing base (5a774e7) to head (8a3b460).

Files with missing lines Patch % Lines
src/course-about/CourseAboutPage.tsx 73.33% 3 Missing and 1 partial ⚠️
src/course-about/utils.ts 87.50% 2 Missing ⚠️
Additional details and impacted files
@@                             Coverage Diff                             @@
##           Peter_Kulko/create-course-about-sidebar      #21      +/-   ##
===========================================================================
- Coverage                                    95.13%   94.53%   -0.61%     
===========================================================================
  Files                                           71       72       +1     
  Lines                                          617      659      +42     
  Branches                                        93      110      +17     
===========================================================================
+ Hits                                           587      623      +36     
- Misses                                          29       33       +4     
- Partials                                         1        3       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@peterkulko

Copy link
Copy Markdown
Author

Reopened at openedx#47

@peterkulko peterkulko closed this Nov 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant