Skip to content

JU-6: add support for openedx-filters#573

Merged
mariajgrimaldi merged 1 commit intoedunext/limonero.masterfrom
li/ednx/JU-6_P3
Nov 25, 2021
Merged

JU-6: add support for openedx-filters#573
mariajgrimaldi merged 1 commit intoedunext/limonero.masterfrom
li/ednx/JU-6_P3

Conversation

@mariajgrimaldi
Copy link
Contributor

@mariajgrimaldi mariajgrimaldi commented Nov 3, 2021

Description

This PR adds support for Open edX Filters in Limonero. These filters are part of the OEP-50: Hooks Extension Framework implementation plan.

For an overall understanding of the design, check out the Open edX Filters ADRs and discussions:

  1. Open edX Filters naming and versioning ADR
  2. Open edX Filters Payload Conventions ADR
  3. Open edX Filters tooling
  4. Other Docs

Supporting information

Discuss on Hooks Extension Framework:
https://discuss.openedx.org/t/configuration-for-the-hooks-extension-framework/4527/
https://discuss.openedx.org/t/hooks-framework-filters/5753

Testing instructions

  1. Checkout to li/ednx/JU-6_P3
  2. Install Open edX Filters library: pip install openedx-filters
  3. Connect your custom actions to the filter (in your site settings):
    "OPEN_EDX_FILTERS_CONFIG": {
        "org.openedx.learning.course.enrollment.started.v1": {
            "fail_silently": false,
            "pipeline": [
                "eox_taylor.XXXX.XXXXX.actions.CanPerformEnrollmentByTag"
            ]
        }
    }
  1. In your course advanced settings, add:
{
    "tags": ["premium"]
}
  1. Install eox-taylor: pip install git+https://bitbucket.org/edunext/eox-taylor/eox-taylor.git@li/ednx/JU-6_P2 #egg=eox-taylor==2__1

  2. Now try to enroll:
    image
    image

@mariajgrimaldi mariajgrimaldi changed the title Li/ednx/ju 6 p3 JU-6: add support for openedx-filters Nov 3, 2021
@mariajgrimaldi mariajgrimaldi force-pushed the li/ednx/JU-6_P2 branch 2 times, most recently from 4fd9486 to b6246c1 Compare November 4, 2021 13:26
@mariajgrimaldi mariajgrimaldi changed the base branch from li/ednx/JU-6_P2 to edunext/limonero.master November 15, 2021 21:05
* Add PreEnrollmentFilter filter before the user's enrollment
@mariajgrimaldi mariajgrimaldi merged commit eaea382 into edunext/limonero.master Nov 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants