fix: return from skuFromRefIds #169
Conversation
|
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
|
node/resolvers/search/index.ts
Outdated
| items = result.map((item: { sellers: any[] }) => ({ | ||
| ...item, | ||
| sellers: item.sellers?.filter((seller: any) => | ||
| 'availability' in seller && | ||
| seller.availability !== null | ||
| ) || [] |
There was a problem hiding this comment.
could you add a comment explaining what/why we are doing here exactly?
ataideverton
force-pushed
the
fix/quickorder
branch
from
8eb7b85
to
96644e3
Compare
|
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 does this PR do? *
Fix the return from getSkuSellerInfo
Before change:
BEFORE.CHANGE.mov
After change:
AFTER.CHANGE.mov
How to test it? *
Try copy/paste skus 880010a (inactive product), 880010v(product not registered) and 880020a(product without stock) on this workspace
Describe alternatives you've considered, if any. *
Related to / Depends on *