Skip to content

fix: derive professor name from DB, not URL - #186

Merged
ZeyadAbbas merged 1 commit into
developfrom
fix/180-professor-name-from-db
Jul 28, 2026
Merged

fix: derive professor name from DB, not URL#186
ZeyadAbbas merged 1 commit into
developfrom
fix/180-professor-name-from-db

Conversation

@SinhSinhAn

@SinhSinhAn SinhSinhAn commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Professor notes page (/notes/JoHn/cOlE) was displaying the raw URL casing instead of the canonical name from the database
  • After fetching sections, the page now pulls profFirst/profLast from the first DB result for display

Closes #180

Test plan

  • Confirm course-only URLs still work normally

@SinhSinhAn
SinhSinhAn requested review from a team as code owners June 22, 2026 03:27
@vercel

vercel Bot commented Jun 22, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
utd-notebook Ready Ready Preview, Comment Jun 22, 2026 3:29am

Request Review

@ZeyadAbbas ZeyadAbbas left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Great work!

@ZeyadAbbas
ZeyadAbbas merged commit bf993ce into develop Jul 28, 2026
3 checks passed
@ZeyadAbbas
ZeyadAbbas deleted the fix/180-professor-name-from-db branch July 28, 2026 13:35
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.

Professor notes page name is derived from URL not from section

2 participants