-
Notifications
You must be signed in to change notification settings - Fork 47
feat: wrap the store around AdsProvider #630
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
base: master
Are you sure you want to change the base?
Conversation
Co-authored-by: Copilot <[email protected]>
|
Hi! I'm VTEX IO CI/CD Bot and I'll be helping you to publish your app! 🤖 Please select which version do you want to release:
And then you just need to merge your PR when you are ready! There is no need to create a release commit/tag.
|
|
Beep boop 🤖 I noticed you didn't make any changes at the
In order to keep track, I'll create an issue if you decide now is not a good time
|
|
Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details. Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
What problem is this solving?
Integrates VTEX Ads into Store Framework via
AdsProviderwrapper. Part of the Ads Frictionless Integration RFC.Key change: ads are now hydrated client-side, enabling CDN cache hits for product data (previously uncacheable due to real-time auction responses).
Changes:
VTEXAdsProvider: wraps store content, provides ad context usingpublisherIdfrom store settings@vtex/ads-reactSDK to request and merge ads with product datapublisherIdsetting to manifest schema (replacesfetchSponsoredProductsOnSearch)How to test it?
Workspace
Search for "camisa" to see sponsored products from an active campaign.
Screenshots or example usage:
Sponsored products displaying with "Patrocinado" badge:
Describe alternatives you've considered, if any.
N/A
Related to / Depends on
How does this PR make you feel? 🔗
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.