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

enterprise An enterprise-related change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant