I have updated SequenceProvider.java so that it fills in all the missing course names and course credits values by grabbing them from the course data DB. This was a necessary change after we decided to exclude these properties from the recommended sequence json data.
One thing that's worth noting is that in the cases where there's an OR in the sequence, as was handled in #38, the frontend will pretend those courses simply don't exist (for now...). I needed to do this to keep things running smoothly.
Now @PeterGhimself, the website is back in a usable state, as you can see on my environment 😄
Sorry I didn't make an issue for this PR, does that matter to you guys?
I have updated
SequenceProvider.javaso that it fills in all the missing course names and course credits values by grabbing them from the course data DB. This was a necessary change after we decided to exclude these properties from the recommended sequence json data.One thing that's worth noting is that in the cases where there's an OR in the sequence, as was handled in #38, the frontend will pretend those courses simply don't exist (for now...). I needed to do this to keep things running smoothly.
Now @PeterGhimself, the website is back in a usable state, as you can see on my environment 😄
Sorry I didn't make an issue for this PR, does that matter to you guys?