Skip to content

DS-228-backport HomepageRenderStarted and CatalogRenderStarted filters #665

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
JuanDavidBuitrago merged 1 commit into from
Sep 6, 2022
Merged

DS-228-backport HomepageRenderStarted and CatalogRenderStarted filters #665

JuanDavidBuitrago merged 1 commit into from
Sep 6, 2022

Conversation

@JuanDavidBuitrago
Copy link
Member

@JuanDavidBuitrago JuanDavidBuitrago commented Sep 3, 2022 •
edited
Loading

Description

This PR make a backport for HomepageRenderStarted and CatalogRenderStarted filters from #659

How to test

  1. Use mango release and use this branch
  2. Install openedx-demo-plugin main branch
  3. Add to your site settings:
"course_org_filters": {
     "demo", ...., "Public"
}
  1. Then create courses using the org "Public". You'll be able to see them on the dashboard but not on the homepage/catalog

@mariajgrimaldi @JuanDavidBuitrago
Merge pull request #659 from eduNEXT/MJG/restriction-filters
2476ed8 
feat: backport HomepageRenderStarted and CatalogRenderStarted filters
@JuanDavidBuitrago JuanDavidBuitrago self-assigned this Sep 4, 2022
@JuanDavidBuitrago JuanDavidBuitrago marked this pull request as ready for review September 4, 2022 03:29
@JuanDavidBuitrago JuanDavidBuitrago force-pushed the JDB/backport_catalog/homepage_filters branch 2 times, most recently from c16ab75 to 2476ed8 Compare September 6, 2022 04:20
@JuanDavidBuitrago JuanDavidBuitrago merged commit 42f55a6 into ednx-release/mango.master Sep 6, 2022
@MaferMazu MaferMazu mentioned this pull request Feb 22, 2023
Closed
@ErickMurillo ErickMurillo mentioned this pull request Feb 12, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Alec4r Alec4r Alec4r approved these changes

@dcoa dcoa Awaiting requested review from dcoa

@MaferMazu MaferMazu Awaiting requested review from MaferMazu

+2 more reviewers

@Henrrypg Henrrypg Henrrypg approved these changes

@danielsgz danielsgz danielsgz approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@JuanDavidBuitrago JuanDavidBuitrago

Labels

backport

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@JuanDavidBuitrago @Henrrypg @Alec4r @danielsgz @mariajgrimaldi