-
Notifications
You must be signed in to change notification settings - Fork 10
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Shopify ScriptTag GA #4103
Shopify ScriptTag GA #4103
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Ignored Deployment
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! Love the example mockups for each event. Just added one suggestion for referring directly to the user instead of calling them "the customer" (especially since we're also referring to Shopify customers as "the customer").
...rtners/message_orchestration/channel_extensions/ecommerce/shopify/shopify_data_processing.md
Outdated
Show resolved
Hide resolved
…erce/shopify/shopify_data_processing.md Co-authored-by: Bre Fitzgerald <82903296+bre-fitzgerald@users.noreply.github.com>
Pull Request/Issue Resolution
Description of Change:
Closes #ISSUE_NUMBER_HERE
Is this change associated with a Braze feature/product release?
✔️ Pull Request Checklist
assets
>js
>broken_redirect_list.js
⭐ Helpful Wiki Shortcuts
❗ ATTN: For PR Reviewers
❗ ATTN: Internal Reviewing Chart
Work at Braze and not sure who to tag for review?
Before tagging @timothy-kim or @KellieHawks for a general review, reference the following chart to see if a specific product vertical/reviewer applies to your pull request.
Platform Infrastructure
Quality Infrastructure
Product Partnerships
SDKs
SMS
FIX
Intelligence
Reporting
SMB
Ingestion
Messaging