Placeholder for AI generated Syllabus Summary Feature 550#551
Placeholder for AI generated Syllabus Summary Feature 550#551
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Few questions
|
|
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? |
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?
I think we should indicate it somewhere in our UI. |
TyHil
left a comment
There was a problem hiding this comment.
Small changes, they could be made into Good first issues if short on time to merge.
|
Oh oops sorry about these, I can work on these and get them done tomorrow morning |
|
tlo
Jing Tao doesn't have a previous syllabus, my bad. It shows a previous summer gov syllabus tho. 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" |
ty!
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 |
|
Yeah might just be better with a slightly different design and as 2 components |
|
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? |
|
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! |
Overview
This is regarding Feature Request 550 "Implement Frontend for Syllabus Parcing. " Created a basic placeholder for frontend.