Skip to content

Add shipping options context #701

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

Open
wants to merge 5 commits into
base: feature/shipping-facets-as-radio
Choose a base branch
from

Conversation

ArthurFerrao
Copy link
Contributor

@ArthurFerrao ArthurFerrao commented Apr 14, 2025

What problem is this solving?

This PR adds the shipping-option-components context, as well as the postal code and pickup modals, which will be opened via the delivery filter on the PLP.
This PR use the deliveryPromisesEnabled value to show or hide delivery filters.

Task

How to test it?

Open and use the postal code and pickup modals through the delivery filter in the PLP.

Workspace

Screenshots or example usage:

Gravacao.de.Tela.2025-04-14.as.18.09.55.mov

Describe alternatives you've considered, if any.

Related to / Depends on

How does this PR make you feel? 🔗

![](put .gif link here - can be found under "advanced" on giphy)

@ArthurFerrao ArthurFerrao requested a review from hiagolcm April 14, 2025 21:12
@ArthurFerrao ArthurFerrao self-assigned this Apr 14, 2025
@ArthurFerrao ArthurFerrao requested review from a team as code owners April 14, 2025 21:12
@ArthurFerrao ArthurFerrao requested review from RodrigoTadeuF, vmourac-vtex and leo-prange-vtex and removed request for a team April 14, 2025 21:12
Copy link
Contributor

vtex-io-ci-cd bot commented Apr 14, 2025

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 😞

@vtex-io-docs-bot
Copy link

Beep boop 🤖

Thank you so much for keeping our documentation up-to-date ❤️

@ArthurFerrao ArthurFerrao force-pushed the feat/shipping-context branch from 437ca30 to a2aec10 Compare April 14, 2025 21:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant