icssc / peterportal-client

A web application to aid UCI students with course discovery and planning.
https://peterportal.org
MIT License
15 stars 12 forks source link

Duplicate Prerequisites #343

Closed KevinWu098 closed 7 months ago

KevinWu098 commented 1 year ago
Screenshot 2023-08-08 at 3 46 23 PM

Stats 67 requires "AP CALCULUS BC" whereas Math 3A requires "AP Calculus BC" resulting in what is essentially duplicate prerequisites which both need to be entered. Will need to investigate further to determine where the error is originating from, but my guess would be that WebSOC is not particularly consistent with how they write their Prereqs...

js0mmer commented 1 year ago

Would have to see if the PPAPI rewrite addresses this. #238 is also kind of related

KevinWu098 commented 1 year ago

Would have to see if the PPAPI rewrite addresses this. #238 is also kind of related

I'll ask @ecxyzzy about it!

KevinWu098 commented 1 year ago

ecxyzzy — 08/08/2023 5:35 PM yeah one of the main goals of the rewrite was to resolve the slew of data inconsistency issues in the old API. I'll probably start migrating it over once the normalized time changes get merged in

From Eddy ^