Skip to content

[DO NOT MERGE] feat!: remove custom header settings#633

Draft
jsnwesson wants to merge 1 commit into
masterfrom
jwesson/remove-custom-header-configuration
Draft

[DO NOT MERGE] feat!: remove custom header settings#633
jsnwesson wants to merge 1 commit into
masterfrom
jwesson/remove-custom-header-configuration

Conversation

@jsnwesson

@jsnwesson jsnwesson commented Jun 3, 2025

Copy link
Copy Markdown
Member

This PR aims to remove the custom header settings for Learner Dashboard. Most of these features already exist as the default experience in frontend-component-header, minus the Programs/Discover New tabs, the Support link, and other 2U-specific features.

@jsnwesson jsnwesson force-pushed the jwesson/remove-custom-header-configuration branch from c4da289 to de1a9d0 Compare June 3, 2025 18:47
@codecov

codecov Bot commented Jun 3, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.47%. Comparing base (9ef5840) to head (3663efb).
⚠️ Report is 104 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #633      +/-   ##
==========================================
- Coverage   97.52%   97.47%   -0.05%     
==========================================
  Files         152      149       -3     
  Lines        1332     1307      -25     
  Branches      229      221       -8     
==========================================
- Hits         1299     1274      -25     
  Misses         32       32              
  Partials        1        1              

☔ 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.

@jsnwesson jsnwesson force-pushed the jwesson/remove-custom-header-configuration branch from de1a9d0 to 3663efb Compare June 3, 2025 18:56
@jsnwesson jsnwesson changed the title feat!: remove custom header settings [DO NOT MERGE] feat!: remove custom header settings Jun 10, 2025
@jsnwesson

Copy link
Copy Markdown
Member Author

Todo: I think there are some messages and track events that can be deleted as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

1 participant