real-time-streams - #203
Conversation
- Created docs/real-time-event-streams/overview.mdx - Created docs/real-time-event-streams/setup.mdx - Created docs/real-time-event-streams/delivery-and-reliability.mdx - Created docs/real-time-event-streams/states-and-monitoring.mdx - Updated docs.json Mintlify-Source: dashboard-editor
- Updated docs/real-time-event-streams.mdx - Updated docs/real-time-event-streams/states-and-monitoring.mdx - Updated docs/real-time-event-streams/delivery-and-reliability.mdx - Updated docs/real-time-event-streams/overview.mdx - Updated docs/real-time-event-streams/setup.mdx - Updated docs.json Mintlify-Source: dashboard-editor
|
@copilot resolve the merge conflicts in this pull request |
Confidence Score: 3/5The PR does not appear safe to merge until the empty group root and contradictory failed-payload retention guidance are corrected. Opening the configured group root still yields no documentation content, and customers receive mutually exclusive guidance about whether failed-event payload samples are available. Files Needing Attention: docs/real-time-event-streams.mdx, docs/real-time-event-streams/states-and-monitoring.mdx, docs/real-time-event-streams/delivery-and-reliability.mdx
|
| Filename | Overview |
|---|---|
| docs.json | Adds the Real-Time Event Streams navigation group and its four documentation pages. |
| docs/real-time-event-streams.mdx | Defines a frontmatter-only group root that still provides no overview content or redirect. |
| docs/real-time-event-streams/delivery-and-reliability.mdx | Documents delivery and retries, but its no-payload-retention contract conflicts with the monitoring page. |
| docs/real-time-event-streams/overview.mdx | Introduces the substantive product overview, beta scope, concepts, and navigation. |
| docs/real-time-event-streams/setup.mdx | Documents destination setup, authentication, filtering, and Liquid payload templates. |
| docs/real-time-event-streams/states-and-monitoring.mdx | Documents operational states and metrics, but incorrectly promises retained payload samples. |
Reviews (3): Last reviewed commit: "Fix title and add description to real-ti..." | Re-trigger Greptile
| --- | ||
| title: "docs/real-time-event-streams" | ||
| --- |
|
|
||
| - **Events delivered** — counts over the last 1 hour, 24 hours, and 7 days. | ||
| - **Delivery success rate** — percentage of events that received a 2xx response within the retry window. | ||
| - **Events failed after max retries** — count, with retained payload samples and destination response. |
There was a problem hiding this comment.
Conflicting payload retention contract
For events that exhaust retries, this page promises retained payload samples while the reliability page explicitly says outbound payloads are not retained, causing customers to rely on unavailable debugging data.
| - **Events failed after max retries** — count, with retained payload samples and destination response. | |
| - **Events failed after max retries** — count, with the destination response. |
Knowledge Base Used: Data activation and pipelines
|
|
||
| {/* PLACEHOLDER: Describe which changes can be made in place (event types, filters, template, auth) and which require pausing the Sync. Clarify how in-flight events are handled during edits. */} | ||
|
|
||
| ## Next steps | ||
|
|
||
| <CardGroup cols={2}> | ||
| <Card title="Delivery and reliability" icon="shield-check" href="/docs/real-time-event-streams/delivery-and-reliability"> | ||
| Retries, failed events log, and at-least-once delivery. | ||
| </Card> | ||
|
|
||
| <Card title="States and monitoring" icon="chart-line" href="/docs/real-time-event-streams/states-and-monitoring"> | ||
| Destination and Sync states, plus how to monitor health. | ||
| </Card> | ||
| </CardGroup> |
There was a problem hiding this comment.
Missing troubleshooting and FAQ
The new documentation ends without the troubleshooting and FAQ material required by the automatically discovered STREAM-310 acceptance criteria, leaving customers without consolidated recovery guidance for common configuration failures.
Source Used: Linear — Public docs
Resolve merge conflict in docs.json by taking the updated structure from main branch. The main branch restructured guides navigation, moving guides from guides-by-use-case/empower-your-team to a flatter structure with better grouping. Since the real-time-streams branch is adding documentation about real-time event streams and not modifying the guides structure, we accept the updated version from main. Co-authored-by: ryancurtis1 <176308989+ryancurtis1@users.noreply.github.com>
✅ Merge conflicts resolved. I've merged the The merge commit has been created and pushed. |
| - **Sync** — a rule that binds a set of filtered events and a Liquid payload template to a Destination. | ||
| - **Failed events log** — a 72-hour record of events that could not be delivered after all retries, retained with the destination HTTP response for debugging. | ||
|
|
||
| <Tip> |
There was a problem hiding this comment.
this is an empty callout in the docs. should add something here or remove
There was a problem hiding this comment.
@copilot can you remove this empty callout please
There was a problem hiding this comment.
Done. Removed the empty callout at line 59 of overview.mdx.
| @@ -0,0 +1,3 @@ | |||
| --- | |||
| title: "docs/real-time-event-streams" | |||
There was a problem hiding this comment.
to the greptile comments point the way this is displayed current should be updated. i don't think this page necessarily needs a bunch of content but we should have something (at least a title instead of the path)
There was a problem hiding this comment.
Done. Replaced the path-based title with "Real-Time Event Streams" and added a proper description to match the pattern of other index pages.
Remove the empty <Tip> block that contained only a placeholder comment at line 59 of docs/real-time-event-streams/overview.mdx. Co-authored-by: ryancurtis1 <176308989+ryancurtis1@users.noreply.github.com>
Replace the path-based title "docs/real-time-event-streams" with a proper customer-facing title and add a description to match the pattern of other index pages in the docs. Co-authored-by: ryancurtis1 <176308989+ryancurtis1@users.noreply.github.com>
Mintlify
11 threads from 1 user in Mintlify