fix(ui): Change modal field to be company size#3801
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
|
@cursor review |
|
@greptile review |
PR SummaryMedium Risk Overview This renames the field throughout the UI schema/form state and the Written by Cursor Bugbot for commit bf0389b. This will update automatically on new commits. Configure here. |
Greptile SummaryThis PR renames the "Number of users" field in the demo-request modal to "Company size" and updates all related constants, types, and the API route consistently. It also refines the size buckets by splitting the old
The rename is thorough and consistent — no stale references to the old Confidence Score: 5/5Safe to merge — small, well-scoped rename with no logic changes or regressions. All three touched files are updated consistently; no database persistence of the old enum values was found, so the bucket rename (1000_plus → 1001_10000 + 10000_plus) carries no backward-compatibility risk. No unrelated code is affected. No files require special attention. Important Files Changed
Sequence DiagramsequenceDiagram
actor User
participant Modal as DemoRequestModal
participant API as /api/demo-requests
participant Email as Email Service
User->>Modal: Fills form (including Company size)
Modal->>Modal: Validates with demoRequestSchema (companySize enum)
Modal->>API: POST { ..., companySize, ... }
API->>API: safeParse with demoRequestSchema
API->>Email: Send email with getDemoRequestCompanySizeLabel(companySize)
Email-->>API: Sent
API-->>Modal: 200 OK
Modal-->>User: "We'll be in touch soon!"
Reviews (2): Last reviewed commit: "Fix lint" | Re-trigger Greptile |
Summary
Rename user count to company size
Type of Change
Testing
Validated modal shows company size
Checklist
Screenshots/Videos