fix: update company type validation to include 'data' and 'ai' types #642
+33
−22
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
bokelley
force-pushed
the
bokelley/fix-company-modal-save
branch
2 times, most recently
from
47cccc8 to
cc68bcf
Compare
The company profile modal on the dashboard was getting stuck on "Saving..." when users selected "Data & Measurement" or "AI & Tech Platforms" because the backend validation only accepted 5 company types (brand, publisher, agency, adtech, other) while the frontend offered 7. Changes: - Update billing-public.ts and organizations.ts to use centralized COMPANY_TYPE_VALUES instead of hardcoded arrays - Add migration to update database CHECK constraint to accept all types - Update admin-tools.ts MCP schema enums to use centralized values 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
bokelley
force-pushed
the
bokelley/fix-company-modal-save
branch
from
cc68bcf to
66dfb47
Compare
bokelley
added a commit
that referenced
this pull request
Jan 6, 2026
…642) The company profile modal on the dashboard was getting stuck on "Saving..." when users selected "Data & Measurement" or "AI & Tech Platforms" because the backend validation only accepted 5 company types (brand, publisher, agency, adtech, other) while the frontend offered 7. Changes: - Update billing-public.ts and organizations.ts to use centralized COMPANY_TYPE_VALUES instead of hardcoded arrays - Add migration to update database CHECK constraint to accept all types - Update admin-tools.ts MCP schema enums to use centralized values 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Fixes the company profile modal getting stuck on "Saving..." when users select "Data & Measurement" or "AI & Tech Platforms" company types.
Root Cause: The frontend modal offered 7 company type options but the backend only validated 5, rejecting 'data' and 'ai'.
Changes
billing-public.tsandorganizations.tsto use centralizedCOMPANY_TYPE_VALUESinstead of hardcoded arraysadmin-tools.tsMCP schema enums to use centralized valuesTest plan
🤖 Generated with Claude Code