icssc / peterportal-client

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

PeterPortal Roadmap showing a black screen after logging in #502

Closed charlieweinberger closed 1 week ago

charlieweinberger commented 2 weeks ago

I went to https://peterportal.org/roadmap/ and began adding classes, as usual. Then, I noticed I could sign in and save my process, so I proceeded to sign in with my UCI email. Once I did so, when I returned to https://peterportal.org/roadmap/, I saw a glimpse of the roadmap as usual, but after half a second or so, I was greeted by a black screen. The continues to happen, even after reloading the page multiple times or signing out and back in again. After checking the devtools console, I see that the error messages below appear at the same time as the page goes black.

Screenshot 2024-11-07 at 11 47 58 PM
Awesome-E commented 2 weeks ago

It looks like this is causing the exception, and it's likely due to part of a recent migration not working: https://github.com/icssc/peterportal-client/blob/main/site/src/pages/RoadmapPage/YearModal.tsx#L26

@js0mmer can you investigate whether quarter names being passed into this function are consistent with existing user roadmaps?

js0mmer commented 2 weeks ago

@Awesome-E I was able to reproduce and put up a PR to fix. #503

js0mmer commented 1 week ago

This should be fixed now, let me know if you still have any issues.