openedx/openedx-platform

Hide sections when a learners cohort doesn't have access to the associated content group.

Open

#32,892 opened on Aug 1, 2023

View on GitHub
 (13 comments) (0 reactions) (1 assignee)Python (4,331 forks)auto 404
enhancementhelp wanted

Repository metrics

Stars
 (8,150 stars)
PR merge metrics
 (PR metrics pending)

Description

@MHaton mentioned that Sections are displayed for learners even when the learners cohort and the sections content group specify it's not for them.

Also annoyingly, learners always have access to sections, even when there are no units assigned to them in those sections. This means you'd have to put something in there saying "This isn't for you, move along". It's not ideal.

@ormsbee thought that the same logic that is used to hide subsections could be extend to hide sections. That logic is here.

Contributor guide

Research direction
Extend the existing subsection visibility logic to sections by modifying the outline generation in cms/djangoapps/contentstore/outlines.py to check cohort access for sections.
Tech stack
python
Domain
backend
Issue type
Feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-2 days
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
GitPython
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
70