Skip to content

Adopt recipient message example guidance for Help reviews - #409

Merged
pelcasandra merged 1 commit into
mainfrom
codex/playbook-message-example-guidance
Oct 11, 2026
Merged

pelcasandra merged 1 commit into
mainfrom
codex/playbook-message-example-guidance

Conversation

@pelcasandra

Copy link
Copy Markdown
Member

Summary

Adopt the merged shared editorial guidance from hellotext/editorial#14 through the pinned docs/editorial submodule.

Every applicable Help review must consider an early recipient message example using the existing message component and an actual supported channel. Useful examples go after the introduction and before configuration instructions, in the article language. An unhelpful example requires a concrete omission reason; useful examples that cannot yet be prepared remain pending.

The rule remains in the canonical shared guide, message guidance and workflow. This consumer change only updates the submodule pin.

Verification

  • The normal Ruby 3.3.6 / Yarn production build and security-header verifier passed.
  • Both assignment article HTML pages and all 24 selected PNGs remain byte-identical to the currently published, reviewed build.
  • docs/ and AGENTS.md remain absent from both language builds.
  • No articles, source screenshots, generated assets, renderer or application state are changed.

@netlify

netlify Bot commented Oct 11, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for legendary-lollipop-e2d131 ready!

Name Link
🔨 Latest commit b4caab0
🔍 Latest deploy log https://app.netlify.com/projects/legendary-lollipop-e2d131/deploys/6acbe0d0f2df750008968e2b
😎 Deploy Preview https://deploy-preview-409--legendary-lollipop-e2d131.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 11, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-11T19:20:20.668934Z b4caab0 Manual request
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@pelcasandra

Copy link
Copy Markdown
Member Author

@codex review

Please review exact head b4caab08eacf44bf0cae6bf094e67c446a40a4ae. The only change is the editorial submodule pin to merged, independently reviewed guidance d0ee357e45d2d14ae605081fc8fe20763b1bb4ba from hellotext/editorial#14. Verify that Help adopts the every-review recipient-example rule, preserves a single canonical guide, and has no unintended article or renderer changes.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Keep it up!

Reviewed commit: b4caab08ea

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@pelcasandra
pelcasandra merged commit 0deceed into main Oct 11, 2026
6 checks passed
@pelcasandra

Copy link
Copy Markdown
Member Author

Adoption verified

Help main now pins editorial d0ee357e45d2d14ae605081fc8fe20763b1bb4ba through normal merge 0deceed305ee799f8f6b24668cd354435dcd064f. The shared guide change was reviewed and merged in hellotext/editorial#14; this exact consumer head was reviewed with no findings and no unresolved threads before its merge.

Main Build 38167653958 passed on the exact merge revision. Normal Netlify production deploy 6acbe1aae89cc000084bf689 is ready and published with that matching revision at 2026-10-11T19:22:07.254Z.

The four canonical guidance files returned HTTP 200 with the expected committed bytes. The existing assignment articles in both languages and all 24 selected PNGs also returned HTTP 200 with unchanged approved hashes. This update changes the review guidance and its pinned consumer version; it does not claim that recipient examples have already been added to existing articles.

Canonical recipient guidance: https://github.com/hellotext/editorial/blob/d0ee357e45d2d14ae605081fc8fe20763b1bb4ba/messages.md#recipient-examples-in-help-articles

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.

1 participant