Skip to content

feat(app-defaults): Add Learning Paths plugin - #4814

Open
debsmita1 wants to merge 3 commits into
redhat-developer:mainfrom
debsmita1:LMS-plugin
Open

debsmita1 wants to merge 3 commits into
redhat-developer:mainfrom
debsmita1:LMS-plugin

Conversation

@debsmita1

@debsmita1 debsmita1 commented Sep 16, 2026

Copy link
Copy Markdown
Member

Hey, I just made a Pull Request!

Resolves:
https://redhat.atlassian.net/browse/RHIDP-16847

Test setup:

Publish a data.json on GitHub with unique card labels (e.g. https://github.com/debsmita1/rhdh/blob/learning-paths-test/packages/app/public/learning-paths/data.json ) so proxy-loaded content is visually distinct from the bundled fallback at packages/app/public/learning-paths/data.json.

app-config.yaml
Point the /developer-hub proxy at the raw GitHub file:

proxy:
  endpoints:
    '/developer-hub':
      target: https://raw.githubusercontent.com/
      pathRewrite:
        '^/api/proxy/developer-hub/learning-paths': '/debsmita1/rhdh/learning-paths-test/packages/app/public/learning-paths/data.json' # configure your path here
      changeOrigin: true
      secure: true

Screenshots:

rhdh-plugins
With proxy configured → cards show proxy labels; network shows /api/proxy/developer-hub/learning-paths → 200

Screenshot 2026-09-16 at 6 20 16 PM

rhdh-local

Screenshot 2026-09-16 at 9 57 15 PM

rhdh-plugins

With proxy disabled/broken → console warns about fallback; network shows /learning-paths/data.json; cards show bundled content.

Screenshot 2026-09-16 at 6 21 19 PM

rhdh-local

Screenshot 2026-09-16 at 10 19 50 PM

Tests:

Screenshot 2026-09-16 at 7 28 47 PM Screenshot 2026-09-16 at 7 30 27 PM

✔️ Checklist

  • A changeset describing the change and affected packages. (more info)
  • Added or Updated documentation
  • Tests for new functionality and regression tests for bug fixes
  • Screenshots attached (for UI changes)

@rhdh-gh-app

rhdh-gh-app Bot commented Sep 16, 2026

Copy link
Copy Markdown

Important

This PR includes changes that affect public-facing API. Please ensure you are adding/updating documentation for new features or behavior.

Changed Packages

Package Name Package Path Changeset Bump Current Version
app workspaces/app-defaults/packages/app none v0.0.0
backend workspaces/app-defaults/packages/backend none v0.0.0
@red-hat-developer-hub/backstage-plugin-app-auth workspaces/app-defaults/plugins/app-auth patch v1.0.1
@red-hat-developer-hub/backstage-plugin-app-defaults workspaces/app-defaults/plugins/app-defaults minor v1.2.1
@red-hat-developer-hub/backstage-plugin-app-integrations workspaces/app-defaults/plugins/app-integrations patch v1.0.1
@red-hat-developer-hub/backstage-plugin-app-react workspaces/app-defaults/plugins/app-react patch v1.2.1

@debsmita1
debsmita1 requested a review from jrichter1 September 16, 2026 14:17
@debsmita1
debsmita1 force-pushed the LMS-plugin branch 2 times, most recently from 46e38da to d671ddc Compare September 16, 2026 14:29
@codecov

codecov Bot commented Sep 16, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 80.41958% with 28 lines in your changes missing coverage. Please review.
✅ Project coverage is 63.16%. Comparing base (86cd1f7) to head (e579ba3).
⚠️ Report is 11 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4814      +/-   ##
==========================================
+ Coverage   63.13%   63.16%   +0.02%     
==========================================
  Files        2661     2669       +8     
  Lines      105935   106073     +138     
  Branches    29621    29658      +37     
==========================================
+ Hits        66884    66999     +115     
- Misses      38525    38548      +23     
  Partials      526      526              
Flag Coverage Δ *Carryforward flag
adoption-insights 84.77% <ø> (ø) Carriedforward from 50daf0d
ai-integrations 78.80% <ø> (ø) Carriedforward from 50daf0d
app-defaults 63.39% <80.41%> (+8.57%) ⬆️
augment 46.67% <ø> (ø) Carriedforward from 50daf0d
boost 84.97% <ø> (ø) Carriedforward from 50daf0d
bulk-import 73.12% <ø> (ø) Carriedforward from 50daf0d
cost-management 13.53% <ø> (ø) Carriedforward from 50daf0d
dcm 73.47% <ø> (ø) Carriedforward from 50daf0d
e2e-adoption-insights 60.00% <ø> (ø) Carriedforward from 50daf0d
e2e-extensions 62.31% <ø> (ø) Carriedforward from 50daf0d
e2e-global-header 49.71% <ø> (ø) Carriedforward from 50daf0d
e2e-homepage 61.11% <ø> (ø) Carriedforward from 50daf0d
e2e-intelligent-assistant 46.08% <ø> (ø) Carriedforward from 50daf0d
e2e-orchestrator 49.49% <ø> (ø) Carriedforward from 50daf0d
e2e-orchestrator-plugin 49.48% <ø> (ø) Carriedforward from 50daf0d
e2e-quickstart 55.21% <ø> (ø) Carriedforward from 50daf0d
e2e-scorecard 50.05% <ø> (ø) Carriedforward from 50daf0d
e2e-theme 16.36% <ø> (ø) Carriedforward from 50daf0d
extensions 58.30% <ø> (ø) Carriedforward from 50daf0d
global-floating-action-button 71.18% <ø> (ø) Carriedforward from 50daf0d
global-header 67.88% <ø> (ø) Carriedforward from 50daf0d
homepage 48.39% <ø> (ø) Carriedforward from 50daf0d
install-dynamic-plugins 71.77% <ø> (ø) Carriedforward from 50daf0d
intelligent-assistant 77.16% <ø> (ø) Carriedforward from 50daf0d
konflux 91.98% <ø> (ø) Carriedforward from 50daf0d
lightspeed 69.02% <ø> (ø) Carriedforward from 50daf0d
mcp-integrations 84.46% <ø> (ø) Carriedforward from 50daf0d
orchestrator 77.32% <ø> (ø) Carriedforward from 50daf0d
quickstart 63.74% <ø> (ø) Carriedforward from 50daf0d
sandbox 79.56% <ø> (ø) Carriedforward from 50daf0d
scorecard 88.48% <ø> (ø) Carriedforward from 50daf0d
theme 87.91% <ø> (ø) Carriedforward from 50daf0d
translations 5.12% <ø> (ø) Carriedforward from 50daf0d
x2a 78.44% <ø> (ø) Carriedforward from 50daf0d

*This pull request uses carry forward flags. Click here to find out more.


Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 86cd1f7...e579ba3. Read the comment docs.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Signed-off-by: Debsmita Santra <[email protected]>
@sonarqubecloud

Copy link
Copy Markdown

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