Skip to content

feat(commerce-onboarding): gate Shopify card behind ?shopify search param - #5743

Merged
viktormarinho merged 1 commit into
mainfrom
viktormarinho/gate-shopify-onboarding
Aug 5, 2026
Merged

feat(commerce-onboarding): gate Shopify card behind ?shopify search param#5743
viktormarinho merged 1 commit into
mainfrom
viktormarinho/gate-shopify-onboarding

Conversation

@viktormarinho

@viktormarinho viktormarinho commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Summary

Hide the Shopify companion card in commerce onboarding (both the blocking connect modal and the /commerce-onboarding route) unless the URL contains a shopify search param (any value: ?shopify, ?shopify=1, …).

The filter lives in useCommerceCompanions, the single point both surfaces build cards from. Filtering the binding requirements up front also means the registry never fetches the Shopify app when the param is absent.

Notes

  • Without the param, an already-linked Shopify connection is also hidden (visual only — the connection itself is untouched).
  • bun run fmt and tsc --noEmit pass.

Summary by cubic

Gate the Shopify companion card in commerce onboarding behind a ?shopify search param. The card won’t render or fetch from the registry unless the param is present.

  • Use useSearch from @tanstack/react-router to show Shopify only when ?shopify is in the URL.
  • Filter companion requirements to skip Shopify when absent, so the registry app isn’t fetched; existing Shopify links remain but are hidden visually.

Written for commit 503eeca. Summary will update on new commits.

Review in cubic

@viktormarinho
viktormarinho merged commit 54563ea into main Aug 5, 2026
25 checks passed
@viktormarinho
viktormarinho deleted the viktormarinho/gate-shopify-onboarding branch August 5, 2026 14:06
decocms Bot pushed a commit that referenced this pull request Aug 5, 2026
PR: #5743 feat(commerce-onboarding): gate Shopify card behind ?shopify search param
Bump type: minor

- decocms (apps/api/package.json): 4.180.1 -> 4.181.0
- @decocms/native (apps/native/package.json): 4.180.1 -> 4.181.0

Deploy-Scope: web
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