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

Dimensões mínimas dos correios ser tratado na lib #31

Closed
ticovix opened this issue Sep 18, 2020 · 2 comments
Closed

Dimensões mínimas dos correios ser tratado na lib #31

ticovix opened this issue Sep 18, 2020 · 2 comments

Comments

@ticovix
Copy link

ticovix commented Sep 18, 2020

image

Olá Lucas! Tive que fazer uma extensão da class Freght, pois o cliente tem produtos com tamanho menor do que os correios pedem. No inicio eu limitava o tamanho dos produtos com o mínimo necessário, porém passei a pensar se isso não teria que ser feito ao calcular todas as dimensões para envio para API. Então passei a limitar as dimensões mínimas para 1, porém quando um cliente adicionava apenas o produto com as dimensões menores no carrinho e calculava o frete, retornava erro na API dos Correios. Para contornar, tive que adicionar esse item pra definir um mínimo na largura, altura e comprimento.

Eu poderia não limitar no cadastro do produto e apenas adicionar os tamanhos mínimos dos correios no banco, porém com a soma da altura e comprimento, isso seria refletido no valor passado para o cliente, certo?

Acha que posso estar fazendo algo errado para ter surgido essa necessidade? Ou acha que isso seria necessário implementar na lib?

@flyingluscas
Copy link
Owner

@ticovix Lembre-se que você não pode considerar apenas as dimensões do produto, precisa considerar as dimensões dele na embalagem que será enviado, por isso o tamanho mínimo dos Correios, se você só calcular usando as dimensões do produto você estará informando para o consumidor um valor de frete que será diferente quando o lojista for despachar o produto. ;)

@flyingluscas
Copy link
Owner

@ticovix Vou fechar a issue mas caso tenha alguma outra dúvida sinta-se a vontade para reabrir ;)

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

No branches or pull requests

2 participants