Skip to content
This repository has been archived by the owner on Mar 23, 2020. It is now read-only.

Pre/co-requisite in API as schema instead of string #78

Open
louy2 opened this issue Dec 3, 2014 · 1 comment
Open

Pre/co-requisite in API as schema instead of string #78

louy2 opened this issue Dec 3, 2014 · 1 comment

Comments

@louy2
Copy link
Contributor

louy2 commented Dec 3, 2014

To do this actually requires the courses to span only one semester, since their pre/co-requisite may change between semesters. Unifying courses from different semesters would require all changed property bound to a start time and an end time, which I think would be a mess to manage.

Proposing such structure:

{
    "prerequisite": [
        {
            "name": "",
            "department_code": "",
            "number": 1010
        }
    ],
    "corequisite": [
        {
            "id": 1,
            "name": "",
            "department_code": "",
            "number": 1010
        }
    ]
}

For co-requisite it is possible to pinpoint the course in the same semester, but not for prerequisite, so id can be added for co-requisite.

@jeffh
Copy link
Owner

jeffh commented Dec 3, 2014

Looks good! Just a small tweak. I think the keys should be pluralized: prerequisite -> prerequisites and corequisite -> corequisites

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants