AA-76: Add is_allowed property to DatesSummary
This takes over some of the logic in the is_enabled property which previously checked two things: (1) is the date time-sensitive and (2) is the date even applicable to this course. Now, is_enabled is only responsible for time-sensitive checks and the new is_allowed property checks applicability. In this way, we can more cleanly separate whether a date block should show up on the dates sidebar (is_enabled and is_allowed) and the dates tab (is_allowed).
Showing
- lms/djangoapps/course_home_api/dates/v1/tests/test_views.py 3 additions, 8 deletionslms/djangoapps/course_home_api/dates/v1/tests/test_views.py
- lms/djangoapps/courseware/courses.py 4 additions, 6 deletionslms/djangoapps/courseware/courses.py
- lms/djangoapps/courseware/date_summary.py 36 additions, 38 deletionslms/djangoapps/courseware/date_summary.py
- lms/djangoapps/courseware/tests/test_date_summary.py 8 additions, 6 deletionslms/djangoapps/courseware/tests/test_date_summary.py
- lms/djangoapps/courseware/tests/test_views.py 6 additions, 3 deletionslms/djangoapps/courseware/tests/test_views.py
- openedx/features/course_experience/tests/views/test_course_home.py 1 addition, 1 deletion...eatures/course_experience/tests/views/test_course_home.py
Loading
Please register or sign in to comment