Conversation
Greptile SummaryThis PR updates webhook event types from simple names (e.g., Key changes:
Issues found:
The implementation is clean and systematic across OpenAPI schemas and webhook endpoint definitions. The changes are breaking and require webhook consumers to update their parsing logic. Confidence Score: 4/5
|
| Filename | Overview |
|---|---|
| openapi/components/schemas/webhooks/WebhookType.yaml | Webhook types updated to dot-notation (e.g., OUTGOING_PAYMENT.COMPLETED). Clean, well-structured change. |
| openapi/components/schemas/webhooks/BaseWebhook.yaml | Added required data field, removed discriminator mapping. Consistent with new webhook envelope structure. |
| openapi/components/schemas/webhooks/CustomerKycWebhook.yaml | New schema replacing KycStatusWebhook. Uses CUSTOMER.* naming pattern with KYC event types. |
| mintlify/snippets/webhooks.mdx | Updated code examples to use type.startsWith() pattern for dot-notation types. Changed webhookId to id. |
| mintlify/ramps/platform-tools/webhooks.mdx | Contains outdated webhook type references (KYC_STATUS, ACCOUNT_STATUS) that need updating to new dot-notation format |
| mintlify/snippets/creating-customers/customers.mdx | Contains old KYC_STATUS webhook example that needs updating to CUSTOMER.KYC_APPROVED format |
Flowchart
%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[Webhook Event] --> B{Old Schema}
B --> C[Single type field<br/>e.g., INCOMING_PAYMENT]
B --> D[Resource-specific fields<br/>transaction, invitation, etc.]
E[Webhook Event] --> F{New Schema}
F --> G[Dot-notation type<br/>e.g., INCOMING_PAYMENT.COMPLETED]
F --> H[Standardized envelope<br/>id, type, timestamp, data]
H --> I[data contains full resource<br/>same as GET endpoint]
J[Migration Impact] --> K[Breaking Change]
K --> L[Consumers must update<br/>type checking logic]
K --> M[Use .startsWith for routing]
K --> N[Parse data field<br/>instead of top-level fields]
style B fill:#ffcccc
style F fill:#ccffcc
style K fill:#ffffcc
Last reviewed commit: 8c37e14
ac110b3 to
3a66bc6
Compare
✱ Stainless preview buildsThis PR will update the kotlin openapi python typescript Edit this comment to update them. They will appear in their respective SDK's changelogs. ✅ grid-openapi studio · code · diff
✅ grid-python studio · code · diff
✅ grid-kotlin studio · code · diff
✅ grid-typescript studio · code · diff
This comment is auto-generated by GitHub Actions and is automatically kept up to date as you push. |
3a66bc6 to
3909f05
Compare
Additional Comments (3)
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
|
3909f05 to
12a7548
Compare
…ation types - Adopt OBJECT.EVENT dot-notation for all webhook types (e.g., OUTGOING_PAYMENT.COMPLETED) - Wrap all webhook data under a unified `data` key (Stripe-style full resource embed) - Rename webhook files to match resource-event pattern - Move test webhook endpoint to /sandbox/webhooks/test - Update documentation and examples to match new schema
12a7548 to
8c37e14
Compare
Additional Comments (5)
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
|

TL;DR
Updated webhook event type names to be more descriptive and consistent.
Changes defined in https://docs.google.com/document/d/1YdRboRlrwm0HKmRdyp-lbHZcfYUtJMF5NWk35vd_dZI/edit?tab=t.0