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

Add course navigation to the 403 forbidden error page #1300

Open
markkuriekkinen opened this issue Oct 30, 2023 · 0 comments
Open

Add course navigation to the 403 forbidden error page #1300

markkuriekkinen opened this issue Oct 30, 2023 · 0 comments
Labels
area: navigation Issues related to menus and navigation effort: days Estimated to take less than one week, from the creation of a new branch to the merging experience: moderate required knowledge estimate O1 needs Requested in the Programming 1 course requester: CS The issue is raised internally by a CS teacher type: feature New feature or change to a feature

Comments

@markkuriekkinen
Copy link
Contributor

When a student is browsing course chapters and opens a chapter that is not yet visible (e.g., because of course module model answer chapter reveal rules), it would be useful if the error page could still include the course navigation: next/previous chapter links and the left-side course menu. Currently, 403 error pages do not include any course-specific navigation.

One potential challenge for the implementation is that the error pages are rendered outside the usual course views. Is it feasible to add any course context there? Note that the 403 error page may also be used in contexts that have nothing to do with a specific course, thus they can't always render course navigation.

https://docs.djangoproject.com/en/4.2/ref/views/#the-403-http-forbidden-view

Started from #1247 (comment)

@markkuriekkinen markkuriekkinen added type: feature New feature or change to a feature O1 needs Requested in the Programming 1 course area: navigation Issues related to menus and navigation effort: days Estimated to take less than one week, from the creation of a new branch to the merging experience: moderate required knowledge estimate requester: CS The issue is raised internally by a CS teacher labels Oct 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: navigation Issues related to menus and navigation effort: days Estimated to take less than one week, from the creation of a new branch to the merging experience: moderate required knowledge estimate O1 needs Requested in the Programming 1 course requester: CS The issue is raised internally by a CS teacher type: feature New feature or change to a feature
Projects
None yet
Development

No branches or pull requests

1 participant