Skip to content

Remove stray customer object from detach-contact request examples - #649

Merged
cox512 merged 1 commit into
mainfrom
jacob.cox/detach-contact-example-fix
Aug 28, 2026
Merged

Remove stray customer object from detach-contact request examples#649
cox512 merged 1 commit into
mainfrom
jacob.cox/detach-contact-example-fix

Conversation

@cox512

@cox512 cox512 commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Why?

The request-body examples for detaching a contact from a group conversation included a customer object. That field isn't part of the request schema and the endpoint ignores it — the contact to detach comes from the URL — so the examples told readers to send something that does nothing, and named a different contact in the body than in the path.

How?

Removes that object from all four request examples on the operation, across Preview and v2.7–v2.16, leaving only the admin id the endpoint actually reads.

Generated with Claude Code

The request-body examples for detaching a contact from a group
conversation included a customer object with an intercom_user_id.
That field is not part of the request schema and the endpoint ignores
it -- the contact to detach is taken from the URL path. The examples
also named a different contact in the body than in the path, which
misled readers into thinking the body identifies the contact.

Removes the object from all four examples on the operation, in every
published version. Examples only; no schema change.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01YXXKa8L8m446cLYBiqtsWF
@cox512
cox512 requested a review from a team August 28, 2026 19:29

@MatthewPelyhes MatthewPelyhes left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Looks good to me. Good catch!

@cox512
cox512 merged commit f8e1baf into main Aug 28, 2026
3 checks passed
@cox512
cox512 deleted the jacob.cox/detach-contact-example-fix branch August 28, 2026 20:06
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.

2 participants