-
Notifications
You must be signed in to change notification settings - Fork 291
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
(Data) Extend the Data Model of Academic Institution to Record Information about Course Materials #2328
Comments
Hi Sabine, I'd like to take up this issue. What are the language requirements for this? |
Hey @0xSaksham, would be happy to have you! This issue is all about making changes to OCaml code. To confirm that the data is available to the templates, you might want to try and render the data to the the EML template |
hi @sabine, I'd like to work on this issue. |
Hi, I can pick this up if it's still open. Looks like there's no extra decoding logic here for now. Is there currently a Dream route serving the |
Hey @csaltachin! Yes, there's a route at https://ocaml.org/academic-users. |
In the Wireframes for the improved "Teaching and Research" page (formerly "Academic Users"), we list course materials from lectures.
To make this truly useful, we need to improve the data model in
academic_institution.ml
:year : int option
tocourse
, so that we can sort the courses by most recent.lecture_notes_available: bool option
to represent whether lecture notes can be found at the URL pointed to by the fieldonline_resource
exercises_available: exercise_availability option
wheretype exercise_availability = Yes | Interactive
(whereYes
maps to the string"yes"
, andInteractive
maps to the string"interactive"
) to represent whether there are exercises provided at the URL pointed to by the fieldonline_resource
.To make this data available in the templates, you need to change
data.mli
and render the new fields to the generated module inacademic_institution.ml
.The task here is only to change the data model and maybe update one data entry (as to keep the PR small).
The text was updated successfully, but these errors were encountered: