search

stumash / CoursePlanner

http://conucourseplanner.online
MIT License
5 stars 3 forks source link

Add some missing courses #111

Closed Davidster closed 7 years ago

Davidster commented 7 years ago

I DID THE SLAVE WORK 🥇

related to issue #86

Summary

Please read the new document I added: missingCoursesNotes.md before continuing. It's really short. The markdown turned out kinda ugly so maybe its better to just view it as plain text when u read it.

To completely fix the issue described in #86, we must accomplish two things on the webscraping level:

  1. Remove ENGR 417 from the sequence(s) it's contained in.
  2. Rename the remainder of the missing courses listed in missingCoursesNotes.md to their true counterpart.

Test

Try to run missingCourses.js and you should only get the courses I documented as problematic in missingCoursesNotes.md.

j a u r a i f o k i n b e s o i n

Davidster commented 7 years ago

Yup exactly what I was thinking. Perhaps I'll make a file courseCorrectionMap.json which maps the changed courses either to their new counterpart or to an empty string to indicate that it should be removed. Those two cases would account for the remainder of the missing courses. What you think?