Course prerequisites #858
Open
Course prerequisites #858
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This will allow for specifiying prerequisites for a course.
The new method allows for retrieval of course details for all outstanding prerequisites a student needs for a session.
This adds a dictionary (keyed by Course id) of Session details to the view's context. These session details are for future sessions being held for prerequisites of the current session. This detail facilitates adding prerequisite buttons to the registration form when a student is missing one or more of the prerequisites required.
The tag will now disable the enroll button when a student does not meet the prerequisites for a session. It will also add buttons for prerequisite course sessions. These buttons will navigate to the registration page of an upcoming session for the prerequisite, if avaialable.
Will now programmatically list prerequisites associated with a class.
This class name was accidentally left in from when I was still playing around with the look of things.
…llcode-website into feature/course-prerequisites
|
@karbassi I have added the prerequisites to the class view templates for the guardian and public views. I didn't think it made sense for the mentor view—don't want to have someone think they can't mentor a class before mentoring for a prerequisite class. If you don't agree with this let me know and I'll add it to the mentor's class view template as well. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
With this PR:
This partially addresses #816. The outstanding item needed to close this out is related to the Courses list on the Programs page, will comment on the issue with details.