Skip to content

Add course search & deadlines pages, Vercel deploy workflow, and repo docs/ignores#16

Merged
DearMrFree merged 1 commit into
mainfrom
codex/locate-vr-enrollment-repository
May 20, 2026
Merged

Add course search & deadlines pages, Vercel deploy workflow, and repo docs/ignores#16
DearMrFree merged 1 commit into
mainfrom
codex/locate-vr-enrollment-repository

Conversation

@DearMrFree
Copy link
Copy Markdown
Owner

Motivation

  • Provide a searchable, filterable Course Catalog and a single-source deadlines page for the 2026–27 catalog cycle to make course discovery and milestone tracking easier.
  • Enable automated Preview and Production deployments to Vercel from GitHub to streamline CI/CD.
  • Improve repository hygiene and onboarding by adding Python ignores and a repository map / deployment notes in README.md.

Description

  • Add a GitHub Actions workflow at .github/workflows/vercel-deploy.yml that creates Vercel Preview deployments for PRs and Production deployments on pushes to main, and expects VERCEL_TOKEN, VERCEL_ORG_ID, and VERCEL_PROJECT_ID secrets.
  • Implement a new deadlines calendar module lib/course-deadlines.ts exporting MIT_COURSEWARE_DEADLINES and sortDeadlinesByDate.
  • Add a deadlines page at app/courses/deadlines/page.tsx that renders the central deadline table and uses sortDeadlinesByDate.
  • Enhance the course catalog in app/courses/page.tsx to support server-side searchParams (page now async) with a text search and honors checkbox filter, hide empty sections, and show a link to the new deadlines page.
  • Update .gitignore to include Python caches, virtualenv and test caches.
  • Expand README.md with a repository map, suggested next work items, and Vercel deployment instructions/secrets.

Testing

  • No automated tests were executed as part of this change.

Codex Task

@vercel
Copy link
Copy Markdown

vercel Bot commented May 20, 2026

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

Project Deployment Actions Updated (UTC)
sofai Ready Ready Preview, Comment May 20, 2026 4:49am

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 593081a620

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread app/courses/page.tsx
searchParams?: Promise<CourseCatalogSearchParams>
}) {
const params = (await searchParams) ?? {}
const query = (params.q ?? "").trim()
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Normalize q before calling .trim()

searchParams values can be either string or string[] in App Router pages, but this code assumes q is always a string. If the URL contains repeated query params (for example /courses?q=math&q=cs), params.q becomes an array and (params.q ?? "").trim() throws a runtime TypeError, causing the page to fail. Please coerce q to a single string (e.g., first element) before trimming.

Useful? React with 👍 / 👎.

@DearMrFree DearMrFree merged commit 5786b4e into main May 20, 2026
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant