search

UWFlow / rmc

Flow is a website that lets you plan courses with friends.
https://uwflow.com
MIT License
233 stars 73 forks source link

Support instructors without commas in their names #329

Closed noryb009 closed 5 years ago

noryb009 commented 5 years ago

It looks like open data (and adm) incorrectly displays one of the instructors for FR192A this semester, only displaying their last name. This led to the data processor throwing an exception, which is probably why the website stopped updating at the beginning of this term (#328).

The attached pull request gives a quick workaround. Using this workaround, l was able to crawl this and next term's sections.

For a slightly better work around, we could hard code their first name based on their last name, and hope this problem doesn't happen again. If we want a permanent work-around, the best source for first names I could find is the UW white pages, which isn't very crawling-friendly.

jswu commented 5 years ago

This looks good to me. Thanks for the fix!