vtex-apps / product-summary

VTEX Product Summary app
11 stars 51 forks source link

move useSimulation call from ProductSummaryPrice to the ProductSummary #280

Closed hiagolcm closed 4 years ago

hiagolcm commented 4 years ago

What problem is this solving?

Currently, the price async is working only when the store is using the ProductSummaryPrice. This PR moves the useSimulation from the ProductSummaryPrice to the ProductSummary. This way, the price async will work in any situation.

mg9CV9NmH7

How to test it?

Workspace

vtex-io-ci-cd[bot] commented 4 years ago

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.

vtex-io-docs-bot[bot] commented 4 years ago

Beep boop :robot:

I noticed you didn't make any changes at the docs/ folder

In order to keep track, I'll create an issue if you decide now is not a good time

vtex-io-ci-cd[bot] commented 4 years ago

Your PR has been merged! App is being published. :rocket: Version 2.62.0 → 2.63.0

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:

vtex deploy vtex.product-summary@2.63.0

After that your app will be updated on all accounts.

For more information on the deployment process check the docs. :book: