Skip to content

Conversation

@wender
Copy link
Contributor

@wender wender commented Aug 27, 2024

What problem is this solving?

Removes workaround to avoid an infinite loop.
This PR depends on https://github.com/vtex/omnishipping/pull/1470

How to test it?

the following credentials
wender.lima@vtex.com
Vtex@123

Make sure you have the Bevi Internal cost center selected.

Add products to the cart https://bevidev.myvtex.com/checkout/cart/add/?sku=10531190&qty=1&seller=1&sc=1

Click to change the address; when you see the list, click Add new Address, don't add any information, just move back to the address list.

Notice that the address list changed.

Now repeat the same using wender workspace https://wender--bevidev.myvtex.com/checkout/cart/add/?sku=10531190&qty=1&seller=1&sc=1

The address will persist the same as before.

@vtex-io-docs-bot
Copy link

vtex-io-docs-bot bot commented Aug 27, 2024

Beep boop 🤖

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

  • There's nothing new to document 🤔
  • I'll do it later 😞

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

  • I just updated 🎉🎉

@vtex-io-ci-cd
Copy link

vtex-io-ci-cd bot commented Aug 27, 2024

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:

  • Patch (backwards-compatible bug fixes)

  • Minor (backwards-compatible functionality)

  • Major (incompatible API changes)

And then you just need to merge your PR when you are ready! There is no need to create a release commit/tag.

  • No thanks, I would rather do it manually 😞

@github-actions
Copy link

Messages
📖 ❤️ Thanks!
📖

🎉 PR additions = 4, PR deletions = 22

Generated by 🚫 dangerJS against 4c1e876

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants