Repository navigation
fix(integrations): validate zendesk, pipedrive, and intercom against vendor specs - #10028
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
[Medium impact] The PR appears safe to merge, with no new actionable issues found. SummaryUpdates Zendesk, Pipedrive, and Intercom tools to match vendor request formats and adds missing block fields.
Reviews (3) · Last reviewed commit: "fix(intercom): label the message recipie..." · Reviewed by Greptile |
There was a problem hiding this comment.
All reported issues were addressed across 54 files
Reply with feedback, questions, or to request a fix.
Turn on auto-fix | Re-trigger cubic
|
@cubic-dev-ai review this PR |
@waleedlatif1 I have started the AI code review. It will take a few minutes to complete. |
There was a problem hiding this comment.
All reported issues were addressed across 56 files
Reply with feedback, questions, or to request a fix.
Turn on auto-fix | Re-trigger cubic
|
@cubic-dev-ai review this PR |
@waleedlatif1 I have started the AI code review. It will take a few minutes to complete. |
ae2e598 to
d12a63c
Compare
…Support API and expose missing block fields
…send lead visible_to as a string, guard path ids
…oozed_until, guard path ids, add user/lead recipient types
…d type, accept boolean done on Pipedrive updates, keep the Pipedrive file_type output
… on create, validate Intercom snooze timestamps, and tighten Pipedrive outputs
d12a63c to
c54e8f8
Compare
Summary
Validated Zendesk, Pipedrive, and Intercom against each vendor's OpenAPI spec.
type:ticketin the/search/exportquery, which the docs say errors, and used undocumentedassignee_id:/organization_id:keywords; now usesfilter[type]alone andassignee:/organization:type:tokens for the same reasontarget_commentas an object; now a string plustarget_comment_is_publicticketIdsfield on the canvas, so it always failed; now exposedorganizationNamedescription/detailsperson_idis sent as a primary participant, anddoneis sent as a booleansort_by/limitand page bycursorvisible_tois sent as the documented string enum, and projects get thecanceledstatusmessage_type: snoozedto/reply, which only accepts comment/note/quick_reply; it now posts to/partswith an integersnoozed_untiluser/leadtypesBehavior notes:
person_idremains, but v1-only fields such asperson_name/org_nameare gone.user(thecontactoption stays for saved workflows).Type of Change
Testing
blockstests,check-block-registry,check:tool-metadata,check:docs,check:docs-manifest, biomeChecklist
test-auditauthoring gate)