Skip to content

Correct Theme Kit checkout upload scope - #53

Merged
next-devin merged 1 commit into
mainfrom
guide-source-corrections
Sep 4, 2026
Merged

Correct Theme Kit checkout upload scope#53
next-devin merged 1 commit into
mainfrom
guide-source-corrections

Conversation

@next-devin

Copy link
Copy Markdown
Contributor

Summary

  • clarify that Theme Kit downloads checkout templates but does not upload them
  • align the source documentation with current ntk push behavior used by the refreshed merchant guides

Validation

  • npm run build
  • npm run check

@cloudflare-workers-and-pages

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
developer-docs 7ceefc3 Commit Preview URL

Branch Preview URL
Sep 04 2026, 04:50 AM

@next-devin
next-devin marked this pull request as ready for review September 4, 2026 04:58
@kilo-code-bot

kilo-code-bot Bot commented Sep 4, 2026

Copy link
Copy Markdown

Code Review Summary

Status: No Issues Found | Recommendation: Merge

The PR is a narrow, accurate documentation correction: the ntk push callout now removes checkout from the upload-eligible directory list and adds an explicit note that ntk pull downloads checkout/ while ntk push silently skips it. Grammar is improved (no "outside of them"), the rest of theme-kit.mdx (including the ntk checkout, ntk watch, and table entries) remains internally consistent, and the change does not introduce contradictions with any unchanged context in the file.

Files Reviewed (1 file)
  • content/docs/storefront/themes/theme-kit.mdx - 1 line changed (line 158), no issues

Reviewed by minimax-m3 · Input: 24.4K · Output: 2.4K · Cached: 147.8K

@next-devin
next-devin merged commit 985d53b into main Sep 4, 2026
3 checks passed
@next-devin
next-devin deleted the guide-source-corrections branch September 4, 2026 05:00
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