icssc / AntAlmanac

A course exploration and scheduling tool for UCI Anteaters
https://antalmanac.com
MIT License
54 stars 64 forks source link

Once PeterPortal API implements past enrollment data, uncomment the graph modal #182

Closed ChaseC99 closed 7 months ago

ChaseC99 commented 3 years ago

AntAlmanac used to store past enrollment data and display it in a modal in the EnrollmentGraph component, rendered inside SectionTable. We removed this feature from our API and requested that PeterPortal API implements it (https://github.com/icssc/peterportal-public-api/issues/111). Once they finish, we should put the modal back but still keep the link to ZotTracker. Reasoning behind moving the data to PeterPortal:

EricPedley commented 2 years ago

We already did this, right? Safe to close?

ChaseC99 commented 2 years ago

We've removed the enrollment data from AntAlmanac but it's not setup on the API yet. You can close this if there's another issue to add the enrollment modal back once it's up on the API

EricPedley commented 2 years ago

So what this issue is talking about is the past enrollment data that used to be stored by us and available in a modal but is now just a link to ZotTracker? Do we have an issue open in PeterPortal API for this?

ChaseC99 commented 2 years ago

Correct. Ideally we bring the modal back and add a "View more on ZotTracker" link on the bottom like how we have with grades.

Here's the API Issue for this task: https://github.com/icssc/peterportal-public-api/issues/111

EricPedley commented 2 years ago

Ok I edited the description of this issue

EricPedley commented 11 months ago

@ecxyzzy when do you think this feature would be ready on the API?