Skip to content
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

Written property to order the means of shipment by the estimated time. #991

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jcamc2811
Copy link

What problem is this solving?

It will be possible to have the ShippingSimulator table sorted by the estimated time for delivery or withdrawal.
New functionality that was requested by the company pay less, thus needing to be adding it to the ShippingSimulator component.

How to test it?

Testing within the paymenus workspace
https://pm650--paguemenos.myvtex.com/cetaphil-hidratante-locao-473ml/p

Preferably use zip code 04662-001 for tests

Screenshots or example usage:

before
image

after
image

Describe alternatives you've considered, if any.

Directly change any code as it was done, or create a new query.

@vtex-io-ci-cd
Copy link
Contributor

vtex-io-ci-cd bot commented Nov 3, 2021

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 ❤️

@jcamc2811 jcamc2811 changed the title Written property for ordering the means of shipping by estimated time. Written property to order the means of shipment by the estimated time. Nov 3, 2021
@jcamc2811 jcamc2811 marked this pull request as ready for review November 8, 2021 12:24
@jcamc2811 jcamc2811 requested review from a team as code owners November 8, 2021 12:24
@jcamc2811
Copy link
Author

jcamc2811 commented Nov 9, 2021

@victorhmp @guerreirobeatriz @icazevedo
Bom dia, poderiam estar verificando minha solicitação para estar adicionando uma nova funcionalidade ao componente ShippingEstimate por favor?

@jcamc2811
Copy link
Author

Oi! Sou o VTEX IO CI / CD Bot e estarei ajudando você a publicar seu aplicativo!🤖

Selecione qual versão você deseja lançar:

  • Patch (correções de bugs compatíveis com versões anteriores)
  • Menor (funcionalidade compatível com versões anteriores)
  • Principal (alterações de API incompatíveis)

E então você só precisa mesclar seu PR quando estiver pronto! Não há necessidade de criar um commit / tag de lançamento.

  • Não, obrigado, prefiro fazer manualmente 😞

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.

None yet

1 participant