Skip to content

Conversation

@SachaProbo
Copy link
Contributor

@SachaProbo SachaProbo commented Jan 2, 2026

Summary by cubic

Make additionalEmailAddresses never null. Default to [] and require it in the API and database.

  • Bug Fixes
    • Default to [] in MCP AddPeopleTool, Console CreatePeople, and n8n Probo node; update GraphQL input to [EmailAddr!]! and add a migration to backfill nulls and enforce NOT NULL.

Written for commit aba5162. Summary will update on new commits.

@SachaProbo SachaProbo force-pushed the fix-people-mcp branch 2 times, most recently from fd273d2 to 7ce5624 Compare January 2, 2026 09:48
@SachaProbo SachaProbo marked this pull request as ready for review January 2, 2026 09:49
Copy link

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 2 files

@SachaProbo SachaProbo force-pushed the fix-people-mcp branch 7 times, most recently from d83c56d to 4fcb9f4 Compare January 5, 2026 10:53
@SachaProbo SachaProbo assigned gearnode and codenem and unassigned gearnode and codenem Jan 5, 2026
@SachaProbo SachaProbo requested review from codenem and gearnode January 5, 2026 14:33
Signed-off-by: Sacha Al Himdani <[email protected]>
@SachaProbo SachaProbo merged commit aba5162 into main Jan 9, 2026
9 checks passed
@SachaProbo SachaProbo deleted the fix-people-mcp branch January 9, 2026 15:28
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.

4 participants