Skip to content

feat: adding CourseEnrollmentViewStarted to execute in enrollment view#363

Open
kiram15 wants to merge 5 commits into
openedx:mainfrom
kiram15:kiram15/ENT-11570
Open

feat: adding CourseEnrollmentViewStarted to execute in enrollment view#363
kiram15 wants to merge 5 commits into
openedx:mainfrom
kiram15:kiram15/ENT-11570

Conversation

@kiram15
Copy link
Copy Markdown
Contributor

@kiram15 kiram15 commented May 28, 2026

Description

This adds a filter CourseEnrollmentViewStarted that will be emitted by the enterprise enrollment view before enrollment is created. We are not reusing the existing CourseEnrollmentStarted filter because this is triggered at the model level, and we want to scoped this to the view-layer enrollment path, not to every programmatic enrollment.

This is part of an ongoing initiative to leverage openedx-filters to allow enterprise to customize logic and remove it from edx-platform code.

Supporting information

Jira ticket - ENT-11570

Testing instructions

Please provide detailed step-by-step instructions for testing this change, including any necessary setup, e.g., additional requirements, plugins, configuration variables, etc, and environment details to ensure the reviewer can test the change.

Deadline

"None" if there's no rush, or provide a specific date or event (and reason) if there is one.

Other information

Include anything else that will help reviewers and consumers understand the change.

  • Any other PRs or issues that should be linked here? Any related PRs?
  • Any special concerns or limitations? For example: deprecations, security, or anything you think should be noted.

Checklists

Check off if complete or not applicable:

Merge Checklist:

  • All reviewers approved
  • Reviewer tested the code following the testing instructions
  • CI build is green
  • Changelog entry added using scriv with short description of the change
  • Documentation updated (not only docstrings)
  • Code dependencies reviewed
  • Fixup commits are squashed away
  • Unit tests added/updated
  • Noted any: Concerns, dependencies, migration issues, deadlines, tickets

Post Merge:

  • Trigger the release workflow to create a new GitHub release.
  • Check new version is pushed to PyPI after tag-triggered build is finished.
  • Delete working branch (if not needed anymore)
  • Upgrade the package in the Open edX platform requirements (if applicable)

@kiram15 kiram15 force-pushed the kiram15/ENT-11570 branch from f11e728 to 5a05934 Compare June 1, 2026 21:46
@kiram15 kiram15 force-pushed the kiram15/ENT-11570 branch 4 times, most recently from b7ca5e2 to e69c264 Compare June 1, 2026 22:54
@kiram15 kiram15 force-pushed the kiram15/ENT-11570 branch from e69c264 to 93a44c7 Compare June 1, 2026 23:14
@kiram15 kiram15 changed the title feat: adding EnterpriseEnrollmentPreProcessor to execute in enrollment view feat: adding CourseEnrollmentViewStarted to execute in enrollment view Jun 3, 2026
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.

1 participant