Repository navigation
Docs update - #236
Docs update#236
Conversation
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughUpdated troubleshooting docs to show a Vite dev-server config that enables the Shopify plugin tunnel ( Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
✅ Deploy Preview for bespoke-flan-5eacfc ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@docs/guide/troubleshooting.md`:
- Around line 125-142: The snippet is missing top-level imports and the closing
braces for the exported config; add imports for defineConfig, shopify and your
Tailwind plugin (referencing defineConfig, shopify, tailwindcss) at the top,
then close the default export object and the file (replace the stray
build.rollupOptions fragment with either a completed build.rollupOptions object
or a comment like "// build.rollupOptions omitted for brevity" if you intend to
continue it) so the file parses and the export default defineConfig({...}) is
properly closed.
🧹 Nitpick comments (1)
docs/guide/troubleshooting.md (1)
135-135: Consider strengthening the myshopify.com CORS pattern.The current regex
/\.myshopify\.com$/doesn't enforce HTTPS or anchor at the start of the origin. Since Shopify stores use HTTPS, consider a more restrictive pattern:- /\.myshopify\.com$/, + /^https:\/\/[^\/]+\.myshopify\.com$/,This ensures:
- Only HTTPS connections are allowed
- Proper start-of-string anchoring
- Prevents potential edge cases with malformed origins
* Add release workflow (barrel#185) * Output modulepreload tags only when configured (barrel#186) * Version Packages * Add vite 7 support (barrel#190) * Version Packages (barrel#191) * Add vite 7 support (barrel#201) * Version Packages (barrel#202) * Add Shopify strict Liquid parser support with 'asset' variable (barrel#208) * Version Packages (barrel#209) * Replace npm tokens with OIDC trusted publishing (barrel#210) * Chrome Local Network Access Update & Tunnel Fix (barrel#211) * Recommended config (barrel#214) * Tunnel fix and cors defaults (barrel#215) * Version Packages (barrel#216) * Run CI with node 24 (barrel#217) * Update references to vite-plugin-shopify-clean (barrel#220) * Fix modulepreload tag generation condition (barrel#223) * Update dependencies (barrel#225) * Update pnpm-lock.yaml to sync with package.json (barrel#226) * Version Packages (barrel#224) * fix allowedHosts placement in vite config file (barrel#235) allowedHosts need to be in server for the Shopify theme editor to display correctly in local environment * Docs update (barrel#236) * Add vite 8 support (barrel#242) * Version Packages (barrel#243) * Mitigating supply chain attacks Bump pnpm to 10.34.5 and fix allowBuilds Clarify allowBuilds comment wording * Bump deps on @shopify/cli-kit Declare Node >=22.12.0 engines on vite-plugin-shopify * Bump deps on vitest * Bump deps on eslint and fix lint errors Enable type-aware ESLint and fix fallout Update typescript version Use import type for Vite types in liquid-no-refresh test Drop Node 20 from CI and fix the version matrix Fix debounce generic breaking page-reload DTS build * Update snapshots * Add release note * Update CI --------- Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Wes Turner-Harris <wes.turner@barrelny.com> Co-authored-by: Jonathan Moore <jonathan@stylehatch.com> Co-authored-by: Maria Carter <maria@driver-digital.com> Co-authored-by: Jade W. <140425997+jadewang425@users.noreply.github.com> Co-authored-by: Miguel Montalvo <miguel.montalvo@meetdomaine.com>
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.