Skip to content
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.

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

Course chapters not accessible in module view #1266

Closed
PasiSa opened this issue Sep 12, 2023 · 0 comments · Fixed by #1267
Closed

Course chapters not accessible in module view #1266

PasiSa opened this issue Sep 12, 2023 · 0 comments · Fixed by #1267
Assignees
Labels
area: user interface User interface issues that are not specifically about navigation or user experience (UX) requester: team The issue is raised by a person inside the A+ developer team type: bug This is a bug

Comments

@PasiSa
Copy link
Contributor

PasiSa commented Sep 12, 2023

In the course module view the sub-chapters are shown as not accessible even though they should, see for example here in A+ manual. This seems to relate to the recent modification that applies revel rules to course chapters, and uses the new is_revealed attribute when deciding on chapter accessibility.

This is particularly harmful in situations when a course module is linked as LTI activity (e.g. from MyCourses), because the chapters in the module have become unaccessible.

@PasiSa PasiSa added the type: bug This is a bug label Sep 12, 2023
@PasiSa PasiSa moved this to Todo in A+ sprints Sep 12, 2023
@markkuriekkinen markkuriekkinen added area: user interface User interface issues that are not specifically about navigation or user experience (UX) requester: team The issue is raised by a person inside the A+ developer team labels Sep 12, 2023
PasiSa added a commit to PasiSa/a-plus that referenced this issue Sep 13, 2023
@PasiSa PasiSa moved this from Todo to Under review in A+ sprints Sep 13, 2023
@PasiSa PasiSa self-assigned this Sep 13, 2023
PasiSa added a commit to PasiSa/a-plus that referenced this issue Sep 18, 2023
PasiSa added a commit to PasiSa/a-plus that referenced this issue Sep 18, 2023
@markkuriekkinen markkuriekkinen added this to the v1.20 - Summer 2023 milestone Sep 19, 2023
markkuriekkinen pushed a commit that referenced this issue Sep 19, 2023
@github-project-automation github-project-automation bot moved this from Under review to Done in A+ sprints Sep 19, 2023
PasiSa added a commit to PasiSa/a-plus that referenced this issue Sep 20, 2023
markkuriekkinen pushed a commit that referenced this issue Sep 20, 2023
murhum1 pushed a commit to murhum1/a-plus that referenced this issue Dec 21, 2023
ihalaij1 pushed a commit to ihalaij1/a-plus that referenced this issue Dec 22, 2023
ihalaij1 pushed a commit to ihalaij1/a-plus that referenced this issue Dec 22, 2023
ihalaij1 pushed a commit to ihalaij1/a-plus that referenced this issue Dec 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: user interface User interface issues that are not specifically about navigation or user experience (UX) requester: team The issue is raised by a person inside the A+ developer team type: bug This is a bug
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants