Skip to content

docs: clarify Everflow test-order S2S postback behavior#12

Merged
brettstirlingbond merged 2 commits into
mainfrom
claude/friendly-brown-f8f0b7
Jun 23, 2026
Merged

docs: clarify Everflow test-order S2S postback behavior#12
brettstirlingbond merged 2 commits into
mainfrom
claude/friendly-brown-f8f0b7

Conversation

@brettstirlingbond

Copy link
Copy Markdown
Contributor

Summary

Updates the Everflow app docs to correct the test-order postback behavior, which was previously documented as an absolute ("Test Orders do not fire an S2S postback").

  • The Enable S2S Postback for test orders setting now exists in the app UI. When enabled (alongside the master Enable Everflow S2S Postback toggle), test orders do fire a conversion postback to Everflow. Reworded the callout (warninginfo) to reflect this.
  • Removed a stray trailing \ from the "Include Tax in Conversion Value" field list item.

Verification

  • npm run validate-links — 0 errors.
  • Rendered locally on the dev server; the callout displays correctly, the /docs/manage/orders/test-orders link resolves (200), no console errors on desktop or mobile.

🤖 Generated with Claude Code

Update the Everflow app docs to reflect the "Enable S2S Postback for
test orders" setting: test orders DO fire an S2S postback to Everflow
when that setting is enabled (with the master postback toggle on).
Also remove a stray trailing backslash from the "Include Tax in
Conversion Value" field list item.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Comment thread content/docs/apps/everflow.mdx Outdated
<Callout type="info">

Note that [**Test Orders**](/docs/manage/orders/test-orders) do not fire an S2S postback to Everflow.
By default, [**Test Orders**](/docs/manage/orders/test-orders) do not fire an S2S postback to Everflow. Enable the **Enable S2S Postback for test orders** setting (with **Enable Everflow S2S Postback** also enabled) if you want test orders to fire a conversion postback to Everflow.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

SUGGESTION: Awkward stutter in the new wording — "Enable the Enable S2S Postback for test orders setting" reads as "Enable the Enable...". Consider rewording to avoid the double verb, e.g. "Turn on Enable S2S Postback for test orders (with Enable Everflow S2S Postback also enabled) to have test orders fire a conversion postback to Everflow."

Also note a casing inconsistency: content/docs/manage/orders/test-orders.mdx:52 documents the same toggle as "Enable S2S postback for test orders" (lowercase "postback"), whereas this doc uses title case. Worth aligning the two pages so the in-product label matches what's published.


Reply with @kilocode-bot fix it to have Kilo Code address this issue.

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jun 23, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
nextcommerce-docs c071804 Commit Preview URL

Branch Preview URL
Jun 23 2026, 03:12 AM

@kilo-code-bot

kilo-code-bot Bot commented Jun 23, 2026

Copy link
Copy Markdown

Code Review Summary

Status: No Issues Found | Recommendation: Merge

The incremental changes since the previous review commit (7b63a42) fully resolve the prior SUGGESTION:

  • content/docs/apps/everflow.mdx:70 — Reworded from "Enable the Enable S2S Postback..." to "Turn on the Enable S2S Postback for test orders setting", eliminating the awkward "Enable the Enable..." stutter.
  • content/docs/manage/orders/test-orders.mdx:52 — Updated toggle label casing from "Enable S2S postback for test orders" to "Enable S2S Postback for test orders", aligning with the label shown in everflow.mdx:70 so both pages reference the same in-product setting name.

Verified label consistency across the docs set — only these two locations reference the toggle, and both now use identical casing and wording. The trailing MDX line-continuation and warninginfo callout change from the prior commit remain in good shape. No new findings on the incremental diff.

Files Reviewed (2 files)
  • content/docs/apps/everflow.mdx
  • content/docs/manage/orders/test-orders.mdx
Previous Review Summary (commit 7b63a42)

Current summary above is authoritative. Previous snapshots are kept for context only.

Previous review (commit 7b63a42)

Status: 1 Issue Found | Recommendation: Address before merge (minor)

Overview

Severity Count
CRITICAL 0
WARNING 0
SUGGESTION 1

The substantive change is correct: removing the trailing \ from the "Include Tax in Conversion Value" bullet fixes an MDX line-continuation bug, and softening the callout from warninginfo with the opt-in note accurately reflects the new app behavior. npm run validate-links confirmation plus a 200 on /docs/manage/orders/test-orders covers the main risk vectors.

Issue Details (click to expand)

SUGGESTION

File Line Issue
content/docs/apps/everflow.mdx 70 "Enable the Enable S2S Postback for test orders setting" stutters ("Enable the Enable..."). The setting label casing also diverges from content/docs/manage/orders/test-orders.mdx:52, which uses "Enable S2S postback for test orders". Recommend rewording to avoid the double verb and aligning the toggle label casing between the two pages so the in-product label matches what's published.
Files Reviewed (1 file)
  • content/docs/apps/everflow.mdx - 1 issue

Fix these issues in Kilo Cloud


Reviewed by minimax-m3 · Input: 23.6K · Output: 1.2K · Cached: 68.7K

Address Kilo review: reword "Enable the Enable S2S Postback..." to
"Turn on the Enable S2S Postback..." and align the setting label
casing on the test-orders page to match the in-product label
("Enable S2S Postback for test orders").

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@brettstirlingbond

Copy link
Copy Markdown
Contributor Author

Fixed in c071804.

  • Stutter: reworded Enable the **Enable S2S Postback for test orders** settingTurn on the **Enable S2S Postback for test orders** setting.
  • Label casing: aligned both pages to the in-product label Enable S2S Postback for test orders (capital P, per the app UI). content/docs/manage/orders/test-orders.mdx:52 previously used lowercase "postback" — updated it to match, so the published label now matches the toggle in the Everflow app settings on both pages.

npm run validate-links → 0 errors.

@brettstirlingbond brettstirlingbond merged commit e4d3c32 into main Jun 23, 2026
3 checks passed
@brettstirlingbond brettstirlingbond deleted the claude/friendly-brown-f8f0b7 branch June 23, 2026 03:15
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