Skip to content

feat: add professor info and RMP ratings to course panel#14

Merged
zaidahmad16 merged 1 commit into
mainfrom
updates&bugs
May 26, 2026
Merged

feat: add professor info and RMP ratings to course panel#14
zaidahmad16 merged 1 commit into
mainfrom
updates&bugs

Conversation

@zaidahmad16

Copy link
Copy Markdown
Owner

When a student clicks a course node, the panel now shows which professors are teaching it in Fall 2026 and Winter 2027, along with their RateMyProfessors rating, difficulty, and "would take again" percentage pulled live via ratemyprofessors-client.

  • backend/timetable_scraper.py: scrapes Carleton Central for both terms and populates a new course_instructors table in Neon
  • GET /courses/{code}/instructors: new endpoint returning instructors grouped by term
  • frontend/app/api/rmp/route.js: Next.js API route that queries RMP for a professor by name, scoped to Carleton (school ID 1420), with in-memory caching
  • CoursePanel.jsx: shows instructor cards with color-coded rating, difficulty, take-again %, and a link to their RMP profile

When a student clicks a course node, the panel now shows which
professors are teaching it in Fall 2026 and Winter 2027, along
with their RateMyProfessors rating, difficulty, and "would take
again" percentage pulled live via ratemyprofessors-client.

- backend/timetable_scraper.py: scrapes Carleton Central for both
  terms and populates a new course_instructors table in Neon
- GET /courses/{code}/instructors: new endpoint returning
  instructors grouped by term
- frontend/app/api/rmp/route.js: Next.js API route that queries
  RMP for a professor by name, scoped to Carleton (school ID 1420),
  with in-memory caching
- CoursePanel.jsx: shows instructor cards with color-coded rating,
  difficulty, take-again %, and a link to their RMP profile
@zaidahmad16 zaidahmad16 merged commit 194bffa into main May 26, 2026
2 checks passed
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.

1 participant