Skip to content

feanil/tutor plugin#29

Merged
feanil merged 4 commits intomainfrom
feanil/tutor_plugin
Apr 16, 2026
Merged

feanil/tutor plugin#29
feanil merged 4 commits intomainfrom
feanil/tutor_plugin

Conversation

@feanil
Copy link
Copy Markdown
Contributor

@feanil feanil commented Apr 9, 2026

  • feat: Add installable tutor plugin for the sample plugin
  • build: Add top-level .gitignore from github/gitignore Python template

feanil and others added 2 commits April 8, 2026 12:39
Replaces the non-functional sample_plugin.py placeholder with a proper
installable Python package under tutor/tutorsampleplugin/.

The plugin:
- Installs openedx-sample-plugin (PyPI) into LMS and CMS images via the
  openedx-dockerfile-post-python-requirements patch
- Installs @openedx/sample-plugin (npm) into all MFE images via the
  mfe-dockerfile-post-npm-install patch (global rather than per-MFE because
  env.config.jsx is a shared template rendered for all MFEs)
- Imports CourseList via mfe-env-config-buildtime-imports
- Configures the org.openedx.frontend.learner_dashboard.course_list.v1 slot
  to hide the default content and insert the sample plugin's CourseList

Gracefully degrades when tutor-mfe is not installed (frontend hooks are
skipped via ImportError guard).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 9, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.23%. Comparing base (483ec2a) to head (353f2e0).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #29   +/-   ##
=======================================
  Coverage   88.23%   88.23%           
=======================================
  Files          13       13           
  Lines         408      408           
  Branches       16       16           
=======================================
  Hits          360      360           
  Misses         40       40           
  Partials        8        8           
Flag Coverage Δ
unittests 88.23% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@feanil feanil requested a review from kdmccormick April 9, 2026 18:39
Renames the package to tutor-contrib-sample-plugin following the
convention for Tutor community plugins.

The tutor plugin shares the version determined by the backend's
python-semantic-release run, passed via SETUPTOOLS_SCM_PRETEND_VERSION
at build time. It is published to PyPI in a parallel job alongside the
backend package.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@feanil feanil merged commit 00d6874 into main Apr 16, 2026
8 checks passed
@feanil feanil deleted the feanil/tutor_plugin branch April 16, 2026 18:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants