Skip to content

Clarify Campaigns App and Delivery Tracking setup - #47

Merged
next-devin merged 3 commits into
mainfrom
apps-index-setup
Sep 28, 2026
Merged

next-devin merged 3 commits into
mainfrom
apps-index-setup

Conversation

@next-devin

@next-devin next-devin commented Sep 28, 2026 •

Copy link
Copy Markdown
Contributor

New stores already have Campaigns App in the dashboard sidebar, but its guide still told merchants to install and pin it. Delivery Tracking is added when a NEXT Payments gateway is created, but its guide told every merchant to install it manually before activation. These pages now describe the current default behavior and what to do when an expected app is missing.

Validation: npm run build (including link and capability checks) and git diff --check passed.

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Sep 28, 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 5399066 Commit Preview URL

Branch Preview URL
Sep 28 2026, 05:26 AM

**Hint** - Pin the Campaigns App to the dashboard sidebar for quick access.

</Callout>
Open **Campaigns App** from the dashboard sidebar. If it is installed but not pinned, open it from the **Apps** menu.

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: The new line 15 says Campaigns App is pinned automatically on new stores, but this paragraph still tells users how to pin it from the Apps menu without explaining who would actually need to do that. With the auto-pin behavior, the manual pin step now only applies to legacy or stores where pinning failed — make that explicit so readers on new stores don't second-guess their dashboard.

Suggested change
Open **Campaigns App** from the dashboard sidebar. If it is installed but not pinned, open it from the **Apps** menu.
Open **Campaigns App** from the dashboard sidebar. On stores created before auto-pin, or if it isn't pinned, open it from the **Apps** menu.

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

@kilo-code-bot

kilo-code-bot Bot commented Sep 28, 2026 •

Copy link
Copy Markdown

Code Review Summary

Status: No Issues Found | Recommendation: Merge

The incremental commit reverts the apps/index.mdx landing page that was added in PATCH 1, removing the file and its entries from content/docs/apps/meta.json and content/docs/meta.json. Only deletions are present in the incremental diff; no added or context lines remain, so no inline comments are applicable per the diff-line rules. Existing pre-incremental docs (Campaigns App install wording, Delivery Tracking billing/setup) are unaffected and were already reviewed.

Files Reviewed (3 files in incremental scope)
  • content/docs/apps/index.mdx - 0 issues (deleted in incremental commit; no inline-commentable lines)
  • content/docs/apps/meta.json - 0 issues (deletion only)
  • content/docs/meta.json - 0 issues (deletion only)
Previous Review Summaries (2 snapshots, latest commit ac3d033)

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

Previous review (commit ac3d033)

Status: No Issues Found | Recommendation: Merge

The previous SUGGESTION on content/docs/apps/campaigns-app/index.mdx line 33 has been resolved by commit ac3d033 — the manual pin instruction was removed entirely, leaving "Open Campaigns App from the dashboard sidebar." combined with the auto-install note at line 15. The previous inline comment is now outdated.

No new issues found on the lines changed in this incremental commit.

Files Reviewed (1 file in incremental scope)
  • content/docs/apps/campaigns-app/index.mdx - 0 issues (lines 15 and 33 updated; auto-pin wording is now consistent)

Previous review (commit c637203)

Status: 1 Issue Found | Recommendation: Address before merge

Overview

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

SUGGESTION

File Line Issue
content/docs/apps/campaigns-app/index.mdx 33 New line 15 says Campaigns App is pinned automatically on new stores, but this Configure section still tells users how to pin from the Apps menu without scoping it to legacy / unpinned stores.
Files Reviewed (5 files)
  • content/docs/apps/index.mdx - 0 issues (new entry page, all linked pages verified to exist)
  • content/docs/apps/campaigns-app/index.mdx - 1 issue (auto-pin vs manual-pin fallback wording)
  • content/docs/apps/delivery-tracking.mdx - 0 issues (manual-install vs auto-install split is consistent)
  • content/docs/apps/meta.json - 0 issues (index added at top of pages list)
  • content/docs/meta.json - 0 issues (apps/index added first under Apps section)

Fix these issues in Kilo Cloud


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

@next-devin next-devin changed the title Add Apps entry page and clarify default installations Clarify Campaigns App and Delivery Tracking setup Sep 28, 2026
@next-devin
next-devin merged commit 03ab2a6 into main Sep 28, 2026
4 checks passed
@next-devin
next-devin deleted the apps-index-setup branch September 28, 2026 05:29
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