Skip to content

Notifications start disabled on new stores - #45

Open
brettstirlingbond wants to merge 3 commits into
mainfrom
docs/notifications-disabled-by-default
Open

brettstirlingbond wants to merge 3 commits into
mainfrom
docs/notifications-disabled-by-default

Conversation

@brettstirlingbond

@brettstirlingbond brettstirlingbond commented Sep 18, 2026

Copy link
Copy Markdown
Contributor

New stores now start with customer notifications turned off, but the docs still told merchants everything was Active by default. This updates the notifications page and 2 related pages so merchants know to turn on the emails they need before going live. Stores created before September 17, 2026 keep their old settings, and the page says so.

···

What changed

  • Support & Notifications: the go-live callout and the Notifications section now say new stores start with customer notifications Disabled (Email Changed and the Admin Order Notification stay on). Adds a list of what to enable, Shop Sync guidance for new and older stores, a note that Upcoming Renewal reminders need the Subscription Renewal Notification on, and the emails that send whatever their setting (Order Invoiced, gift card delivery, payment collected, dashboard resends).
  • Shop Sync: the go-live callout and the setup note now say "keep off, or disable on older stores" instead of "disable". The Refund and subscription list now says "enable".
  • Subscription Settings: the Upcoming Renewal dependency, and a note that subscription notifications start disabled.

The source is the NEXT Mind store setup settings card (Notifications section and Gotchas).

Checks

  • npm run build passes.
  • The #notifications and #notifications-settings anchors are unchanged. 9 other pages link to them.
  • Local pre-review: the Codex adversarial pass came back clean on round 2, after adding the rollout date. The MiniMax pass didn't run because the plan hit its usage limit.

🤖 Generated with Claude Code

brettstirlingbond and others added 2 commits September 18, 2026 16:57
Update Support & Notifications, Shop Sync and Subscription Settings to
reflect that new stores ship customer notifications disabled (except
Email Changed and Admin Order Notification). Lists what to enable,
keeps disable guidance for older stores, notes the renewal reminder
dependency and the emails that send regardless of the toggle.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Sep 18, 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 6dc303c Commit Preview URL

Branch Preview URL
Sep 18 2026, 08:31 AM

Comment thread content/docs/apps/shop-sync.mdx Outdated
Comment thread content/docs/apps/shop-sync.mdx Outdated
Comment thread content/docs/start-here/get-started/support-and-notifications.mdx Outdated
Comment thread content/docs/start-here/get-started/support-and-notifications.mdx Outdated
Comment thread content/docs/start-here/get-started/support-and-notifications.mdx Outdated
@kilo-code-bot

kilo-code-bot Bot commented Sep 18, 2026

Copy link
Copy Markdown

Code Review Summary

Status: No Issues Found | Recommendation: Merge

All issues from the previous review have been resolved in the new commits. The incremental changes (commits adding the September 17, 2026 date, splitting new-store/older-store guidance, qualifying the Order Confirmation bullet for Shop Sync, and rewording the bypassed-notification sentence) correctly address the previously flagged WARNING and SUGGESTION items without introducing regressions.

Files Reviewed (3 files)
  • content/docs/apps/shop-sync.mdx - 0 issues
  • content/docs/start-here/get-started/support-and-notifications.mdx - 0 issues
  • content/docs/manage/subscriptions-guide/subscription-settings.mdx - 0 issues
Previous Review Summary (commit ad9f83f)

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

Previous review (commit ad9f83f)

Status: 5 Issues Found | Recommendation: Address before merge

Overview

Severity Count
CRITICAL 0
WARNING 1
SUGGESTION 4
Issue Details (click to expand)

WARNING

File Line Issue
content/docs/start-here/get-started/support-and-notifications.mdx 27 Recommends enabling Order Confirmation, but the Shop Sync note at line 33 says keep matching NEXT notifications off — conflicting advice for Shop Sync merchants.

SUGGESTION

File Line Issue
content/docs/apps/shop-sync.mdx 15 "keep off" + "Disable them if Active" mixes state and imperative for two store populations; hard to read for older-store merchants.
content/docs/apps/shop-sync.mdx 134 "can leave the notifications Shopify sends turned off" is missing "that" and the verbs diverge from the equivalent phrasing on support-and-notifications.mdx:33.
content/docs/start-here/get-started/support-and-notifications.mdx 21 Older stores' previous default (all notifications Active) is implied but never stated, leaving "review the list and Disable" guidance without context.
content/docs/start-here/get-started/support-and-notifications.mdx 35 "gift card delivery" and "payment collected" use informal descriptions rather than the formal notification names used elsewhere in the paragraph.
Files Reviewed (3 files)
  • content/docs/apps/shop-sync.mdx - 2 issues
  • content/docs/start-here/get-started/support-and-notifications.mdx - 3 issues
  • content/docs/manage/subscriptions-guide/subscription-settings.mdx - 0 issues

Fix these issues in Kilo Cloud


Reviewed by minimax-m3 · Input: 0 · Output: 0 · Cached: 0

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
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