Skip to content

docs: documents chat message replies - #1587

Merged
meryldakin merged 3 commits into
mainfrom
codex/document-chat-replies
Aug 19, 2026
Merged

docs: documents chat message replies#1587
meryldakin merged 3 commits into
mainfrom
codex/document-chat-replies

Conversation

@meryldakin

@meryldakin meryldakin commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Description

  • Add chat reply documentation under the Chat integrations section using the current workflow-builder labels.
  • Document replies to previous chat steps and existing Slack or Discord threads.
  • Explain provider-specific parent message IDs, bot-based connection requirements, Slack destination fallback, and metadata.external_id usage.
  • Clarify parent-step eligibility across branches and when a step produces multiple messages.
  • Add a Slack trigger-data example, troubleshooting guidance, sidebar navigation, and links from related documentation.

Validation

  • yarn format.check
  • yarn lint
  • yarn type-check
  • git diff --check origin/main...HEAD

@vercel

vercel Bot commented Aug 5, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview Aug 19, 2026 6:43pm

Request Review

@cursor cursor Bot 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.

Risk MEDIUM: Adds documentation for replying to Slack chat messages, including a new page, sidebar entry, and cross-links from related docs.

Reasons

  • Adds a new MDX page (content/designing-workflows/replying-to-chat-messages.mdx, 100 lines).
  • Updates sidebar navigation in data/sidebars/platformSidebar.ts to surface the new page.
  • Changes span multiple documentation sections (designing-workflows and integrations/chat/slack).
  • Content-only changes with no component, dependency, or build configuration modifications.
  • Author meryldakin is a known team member; approval withheld because risk is MEDIUM, not LOW.

Notes

  • Verify the new page follows Knock style guide (sentence case, "enable" vs "allow", active voice).
  • Confirm cross-links resolve correctly: /designing-workflows/replying-to-chat-messages and anchor #reply-to-an-external-message.
  • Review Liquid and JavaScript code examples for accuracy (parent message ID, channel ID, trigger payload).
  • Check the troubleshooting table for technical correctness against Slack reply behavior.
  • Confirm sidebar placement under "Designing workflows" is the intended navigation location.
Open in Web View Automation 

Sent by Cursor Automation: Docs PR classifier

@cellomatt cellomatt left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Approved, with a bunch of mostly-small nits.

Main thing that we should change in my opinion is moving this documentation under Integrations > Chat, which will also require updating all of the links that reference the new page.

Comment thread data/sidebars/platformSidebar.ts Outdated
Comment thread content/integrations/chat/slack/sending-an-internal-message.mdx Outdated
Comment thread content/designing-workflows/replying-to-chat-messages.mdx Outdated
Comment thread content/designing-workflows/replying-to-chat-messages.mdx Outdated
Comment thread content/designing-workflows/replying-to-chat-messages.mdx Outdated
Comment thread content/designing-workflows/replying-to-chat-messages.mdx Outdated
Comment thread content/designing-workflows/replying-to-chat-messages.mdx Outdated
Comment thread content/designing-workflows/replying-to-chat-messages.mdx Outdated
Comment thread content/designing-workflows/replying-to-chat-messages.mdx Outdated
Comment thread content/designing-workflows/replying-to-chat-messages.mdx Outdated
@meryldakin
meryldakin merged commit b949427 into main Aug 19, 2026
4 checks passed
@meryldakin
meryldakin deleted the codex/document-chat-replies branch August 19, 2026 18:58
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