Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add LARC tutorials as sections #432

Open
EricPedley opened this issue Sep 9, 2022 · 3 comments · May be fixed by #1068
Open

Add LARC tutorials as sections #432

EricPedley opened this issue Sep 9, 2022 · 3 comments · May be fixed by #1068
Assignees
@KevinWu098
Labels
enhancement Improvements to the user experience

Comments

@EricPedley
Copy link
Member

This seems tough because we're gonna have to scrape the LARC site since it's server-side rendered (https://enroll.larc.uci.edu). We might want to split this change up between us and PeterPortal API since the backend stuff is more their domain.

As for the frontend, I'm thinking maybe we have a toggle in the search menu to show LARC tutorials, and if it's toggled on, the LARC tutorials show up inline next to the lectures and discussions in the search results and behave just like any other section. This will be easier to pull off once we merge the TypeScript rewrite and the fields necessary for a section are well-defined.
@EricPedley EricPedley added the enhancement Improvements to the user experience label Sep 9, 2022
@EricPedley EricPedley moved this to Icebox 🥶 in AntAlmanac Sep 9, 2022
@EricPedley EricPedley changed the title Add LARC tutorials to class search Add LARC tutorials Sep 9, 2022
@EricPedley EricPedley changed the title Add LARC tutorials Add LARC tutorials as sections Sep 9, 2022
@EricPedley EricPedley moved this from Icebox 🥶 to Backlog 🥱 in AntAlmanac Sep 9, 2022
@ecxyzzy ecxyzzy mentioned this issue Sep 9, 2022
Open
@ap0nia
Copy link
Collaborator

ap0nia commented Oct 12, 2023

I implemented this for PPA https://docs.api-next.peterportal.org/docs/developers-guide/rest-api/reference/larc

@github-actions github-actions bot added the Stale label Jan 11, 2024
@MinhxNguyen7 MinhxNguyen7 moved this from Icebox 🥶 to Backlog 🥱 in AntAlmanac Nov 14, 2024
@MinhxNguyen7
Copy link
Member

We'll show LARC in the search pane if there is only one course (i.e., user specifically searched for a class).
We'll have to work out whether and how to display unenrolled LARC sections from the added pane.

@MinhxNguyen7
Copy link
Member

Blocked by AAPI bug

@KevinWu098 KevinWu098 linked a pull request Dec 6, 2024 that will close this issue
Draft
4 tasks
@MinhxNguyen7 MinhxNguyen7 moved this from Selected for Development to In Progress 🤠 in AntAlmanac Dec 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@KevinWu098 KevinWu098

Labels
enhancement Improvements to the user experience
Projects
AntAlmanac
Status: In Progress 🤠
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: integrate larc icssc/AntAlmanac
4 participants
@EricPedley @MinhxNguyen7 @ap0nia @KevinWu098