Repository navigation
[STR-1307] fix: add ViewProduct policy for catalog-graphql - #228
Conversation
Co-authored-by: Cursor <cursoragent@cursor.com>
|
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
|
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
|
Your PR has been merged! App is being published. 🚀 After the publishing process has been completed (check #vtex-io-releases) and doing A/B tests with the new version, you can deploy your release by running:
After that your app will be updated on all accounts. For more information on the deployment process check the docs. 📖 |

What problem is this solving?
From
catalog-graphql1.108 on, the app token must be authorized withView*policies. The return-app callscatalog-graphql(skuquery, to translate SKU names) withctx.authTokenbut only declaredvtex.catalog-graphql:resolve-graphql. Same fix applied in broadcaster-worker: vtex-apps/broadcaster-worker#36Only
ViewProductis added, since the query reads only the SKUname(no brand/category).How to test it?
Link the branch and open a flow that triggers SKU name translation; confirm no 403 from
catalog-graphql.[Workspace](Link goes here!)
Screenshots or example usage:
N/A
Describe alternatives you've considered, if any.
If
ViewProductis not enough, also addViewBrandandViewCategoryas in broadcaster-worker.Related to / Depends on
vtex-apps/broadcaster-worker#36