Skip to content

Placeholder for AI generated Syllabus Summary Feature 550#551

Merged
egsch merged 32 commits intodevelopfrom
Feature550
Apr 2, 2026
Merged

Placeholder for AI generated Syllabus Summary Feature 550#551
egsch merged 32 commits intodevelopfrom
Feature550

Conversation

@barkat-10
Copy link
Copy Markdown
Contributor

@barkat-10 barkat-10 commented Dec 1, 2025

Overview

This is regarding Feature Request 550 "Implement Frontend for Syllabus Parcing. " Created a basic placeholder for frontend.

@barkat-10 barkat-10 requested a review from egsch as a code owner December 1, 2025 08:20
@vercel
Copy link
Copy Markdown

vercel Bot commented Dec 1, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
utd-trends Ready Ready Preview, Comment Apr 2, 2026 3:11pm

Request Review

@AbhiramTadepalli
Copy link
Copy Markdown
Contributor

Few questions

  • Any modifications we want to make the the prompt / is the summary good & representative of the prof's course?
  • Can we / should we indicate which semester the syllabus is from?
  • Should this be added to planner?
  • How long should we cache a syllabus? Every semester we get a new link, so potentially we cache it forever? I also think like a 30 day limit is chill (in case we update prompts. If a professor updates the syllabus can they upload the new doc at the same link?)
  • I think the rmp ai + syllabus ai ui can be modified by designers in the Spring, but we can still probably merge this in in the meantime

@egsch
Copy link
Copy Markdown
Contributor

egsch commented Mar 30, 2026

Added some UI changes and an attempt to fix the useEffect warnings and potential loop. Not totally sure the second fix works, but it seems to so far.

Are we ready to enable caching?

@AbhiramTadepalli
Copy link
Copy Markdown
Contributor

AbhiramTadepalli commented Mar 30, 2026

Should this be added to planner?

Right now you can access the syllabus summary as you would in the dashboard. There are also "View Syllabus" links next to each section. Is this what we want?

Can we / should we indicate which semester the syllabus is from?

I think we should indicate it somewhere in our UI.

Copy link
Copy Markdown
Member

@TyHil TyHil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small changes, they could be made into Good first issues if short on time to merge.

Comment thread src/app/api/syllabusSummary/route.ts Outdated
Comment thread src/app/api/syllabusSummary/route.ts
Comment thread src/components/planner/PlannerCoursesTable/PlannerCard.tsx Outdated
Comment thread src/components/search/SearchResultsTable/SearchResultsTable.tsx Outdated
@AbhiramTadepalli
Copy link
Copy Markdown
Contributor

AbhiramTadepalli commented Mar 31, 2026

  • remove the dropdown when no syllabus exists
  • fix: GOVT 2306 Jing Tao (no grades and/or no rmp) has a previous syllabus, but the component doesn't show (Summer 2026 Planner)
  • Syllabus Summary now showing in MyPlanner (it used to in cbe70d5)

@egsch
Copy link
Copy Markdown
Contributor

egsch commented Mar 31, 2026

  • remove the dropdown when no syllabus exists

  • fix: GOVT 2306 Jing Tao (no grades and/or no rmp) has a previous syllabus, but the component doesn't show (Summer 2026 Planner)

  • Syllabus Summary now showing in MyPlanner (it used to in cbe70d5)

Oh oops sorry about these, I can work on these and get them done tomorrow morning

@AbhiramTadepalli
Copy link
Copy Markdown
Contributor

tlo

[ ] fix: GOVT 2306 Jing Tao (no grades and/or no rmp) has a previous syllabus, but the component doesn't show (Summer 2026 Planner)

Jing Tao doesn't have a previous syllabus, my bad. It shows a previous summer gov syllabus tho.
The underlying edge case is if a prof has no RMP, their syllabus summary won't be show as <SingleProfInfo/> isn't rendered.

I think that's fine for now?

Copy link
Copy Markdown
Contributor

@AbhiramTadepalli AbhiramTadepalli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGMT!

@TyHil
Copy link
Copy Markdown
Member

TyHil commented Apr 2, 2026

Wouldn't it be pretty easy to just separate the syllabus summary from the RMP component? Seems odd that they're coupled anyways

Ig if you're just trying to merge, that could become a "good first issue"

@egsch
Copy link
Copy Markdown
Contributor

egsch commented Apr 2, 2026

Jing Tao doesn't have a previous syllabus, my bad. It shows a previous summer gov syllabus tho.

ty!

The underlying edge case is if a prof has no RMP, their syllabus summary won't be show as <SingleProfInfo/> isn't rendered. I think that's fine for now?

Wouldn't it be pretty easy to just separate the syllabus summary from the RMP component? Seems odd that they're coupled anyways

Ig if you're just trying to merge, that could become a "good first issue"

yeah that's probably a good idea. the only issue is that rn in the UI we have a single row with RMP link & syllabus summary dropdown which is kinda awkward to recreate in 2 components. I had a version where the component was universally shown and we had a check inside to only show RMP info if ncecessary, but I can try to do it as 2 components instead

@TyHil
Copy link
Copy Markdown
Member

TyHil commented Apr 2, 2026

Yeah might just be better with a slightly different design and as 2 components

@AbhiramTadepalli
Copy link
Copy Markdown
Contributor

How about we merge (since it's an edge case that only happens when the prof doesn't have rmp (they're new, or no one rly cares to review)) and address it in the expanded view redesign?

@TyHil
Copy link
Copy Markdown
Member

TyHil commented Apr 2, 2026

Sounds good. I think if you can make a quick issue rn tho, just so it's a documented bug, that'd be good

@egsch
Copy link
Copy Markdown
Contributor

egsch commented Apr 2, 2026

Sounds good. I think if you can make a quick issue rn tho, just so it's a documented bug, that'd be good

just made an issue!

@egsch egsch merged commit c7188c1 into develop Apr 2, 2026
3 checks passed
@egsch egsch deleted the Feature550 branch April 2, 2026 23:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implement frontend for syllabus processing

4 participants