Skip to content

feat: replace enterprise_support import with DiscountEligibilityCheckRequested filter#38099

Draft
pwnage101 wants to merge 1 commit intomasterfrom
pwnage101/ENT-11564
Draft

feat: replace enterprise_support import with DiscountEligibilityCheckRequested filter#38099
pwnage101 wants to merge 1 commit intomasterfrom
pwnage101/ENT-11564

Conversation

@pwnage101
Copy link
Contributor

@pwnage101 pwnage101 commented Mar 4, 2026

Removes direct lazy imports of is_enterprise_learner from openedx.features.enterprise_support.utils in applicability.py and replaces them with calls to the DiscountEligibilityCheckRequested openedx-filter. Adds the filter to OPEN_EDX_FILTERS_CONFIG with the DiscountEligibilityStep pipeline step. Updates tests to mock the filter instead of using enterprise model factories.

ENT-11564

Co-Authored-By: Claude Sonnet 4.6 noreply@anthropic.com


Blocked by:

…Requested filter

Removes direct lazy imports of is_enterprise_learner from
openedx.features.enterprise_support.utils in applicability.py and
replaces them with calls to the DiscountEligibilityCheckRequested
openedx-filter. Adds the filter to OPEN_EDX_FILTERS_CONFIG with the
DiscountEligibilityStep pipeline step. Updates tests to mock the filter
instead of using enterprise model factories.

ENT-11564

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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