Skip to content

Conversation

@gpalmer27
Copy link
Collaborator

Description

what it says in the title above. also fixed bug with company popup where pay amounts were appearing multiple times!

Motivation and Context

Closes #344

How has this been tested?

created several new reviews and made sure it worked for both coop and internship and that nothing broke anywhere the .jobtype was originally referenced

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Database migration
    • Ran pnpm db:generate and verified generated SQL migration files in packages/db/drizzle

Checklist:

  • My code follows the code style of this project.
  • I have moved the ticket to "In Review"
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@vercel
Copy link

vercel bot commented Feb 8, 2026

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

Project Deployment Actions Updated (UTC)
cooper Ready Ready Preview, Comment Feb 9, 2026 3:46am
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
cooper-auth Skipped Skipped Feb 9, 2026 3:46am
cooper-docs Skipped Skipped Feb 9, 2026 3:46am

Request Review

key={model.name}
className={`h-full rounded-lg ${
model.name === "In-person"
model.name === "Inperson"
Copy link
Contributor

Choose a reason for hiding this comment

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

why no dash in between

@gpalmer27
Copy link
Collaborator Author

Should show 'co-op/internship' in role preview + role info card

@vercel vercel bot temporarily deployed to Preview – cooper-auth February 8, 2026 21:03 Inactive
@vercel vercel bot temporarily deployed to Preview – cooper-docs February 8, 2026 21:03 Inactive
Copy link
Contributor

@leeyandrea leeyandrea left a comment

Choose a reason for hiding this comment

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

lgtm

@vercel vercel bot temporarily deployed to Preview – cooper-auth February 9, 2026 03:45 Inactive
@vercel vercel bot temporarily deployed to Preview – cooper-docs February 9, 2026 03:45 Inactive
@songmichael11 songmichael11 merged commit 5eeeda1 into main Feb 9, 2026
10 checks passed
@songmichael11 songmichael11 deleted the change-jobtype branch February 9, 2026 04:31
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.

Refactor schema to make JobType field belong to reviews instead of roles table

4 participants