fix: content proposal broken due to missing title column in users table#1295
Merged
Conversation
…itle) The users table has never had a title column, causing all content submissions via propose_content to fail with "column 'title' does not exist". Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Fix logo not rendering on profile cards and member directory: hosted brand.json may have logos at the top level (simple format) rather than nested under brands[0] (house_portfolio). Apply the same brands[0] -> top-level fallback pattern everywhere brand data is read from hosted_brands (http.ts x3, member-profiles.ts). - Fix has_manifest false negative in resolve_brand tool: when resolveBrand() returns null (e.g. AAO-hosted brands whose authoritative_location pointer chain doesn't resolve via validateDomain), check hosted_brands as a fallback before discovered brands. Hosted brands have a real manifest so has_manifest is always true. - Fix authoritative_location redirect in BrandManager: use hostname only when following the redirect (was using hostname + pathname, creating nonsensical URLs that validateDomain can't handle). - Fix Set up brand link going to dashboard instead of brand setup: existing members sent to /onboarding?step=brand get redirected to /dashboard by the onboarding flow's hasCompanyOrgs guard. Link now goes directly to /brand/builder which is the correct tool. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Revert brand-manager.ts authoritative_location change — it does not fix the redirect chain for hosted brands (still returns null). The fallback in brand-tools.ts is the correct fix; the manager change only added a misleading comment. - Fix brand-tools.ts comment: "Check hosted brands first" → "before falling back to the discovered-brand registry" - Remove dead authorTitle = undefined in content.ts — replace with inline null at the two call sites Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
bokelley
force-pushed
the
bokelley/addie-escalation-cleanup
branch
from
daa7a34 to
764a359
Compare
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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
propose_contentalways failing withcolumn 'title' does not existgetUserInfofunction incontent.tswas selectingtitlefrom theuserstable, but that column was never added to the schemaRoot cause
getUserInfoqueriedSELECT first_name, last_name, title, email FROM usersbut theuserstable (migration079_users_table.sql) has notitlecolumn.Test plan
POST /api/content/proposenow returns 201 instead of 500🤖 Generated with Claude Code