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

Validação de Peso e Dimensões #43

Closed
jarjar123 opened this issue Dec 15, 2014 · 3 comments · Fixed by #47
Closed

Validação de Peso e Dimensões #43

jarjar123 opened this issue Dec 15, 2014 · 3 comments · Fixed by #47
Milestone

Comments

@jarjar123
Copy link
Contributor

Gostaria de apontar alguns problemas quanto as validações de peso e tamanho.

Acredito que poucos habilitem esse recurso, pois produtos com dimensões e/ou peso inferiores ao limite mínimo acusam erro, e invalidam a cotação. E também não considera quantidades. Ao adicionar 100 DVDs no carrinho, somente a dimensão de 1 unidade será validada, podendo acusar erro pelos limites inferiores.

Além disso a validação considera limites iguais para todos os serviços. E apesar da maioria ser igual, existem exceções como eSedex (até 15kg), PAC GF (até 3m), Carta Registrada (até 500gr), etc.
Mas são as exceções que trazem benefícios financeiros, e a maioria dos logistas querem explorá-las.

Enfim fica a sugestão de validar o carrinho como um todo (e não apenas o produto), e também considerar o serviço que será cotado, antes de realizar a validação. Essa melhoria é o alicerce para adição do PAC GF, e a inclusão de métodos não contemplados pelo webservice.

@pedro-teixeira
Copy link
Owner

Tivemos uma boa discussão sobre esse tópico no #3

@jarjar123
Copy link
Contributor Author

Ok, visto o tópico #3, deixo aqui estas sugestões, quanto a validação dos limites de peso e tamanho.

  1. Validar os itens do carrinho considerando as quantidades
  2. Validar também o carrinho todo
  3. Não validar os limites inferiores
  4. Especificar limites diferentes para cada serviço de postagem
  5. Enviar o peso real e a dimensão média

@pedro-teixeira
Copy link
Owner

Ficamos aguardando uma PR

jarjar123 pushed a commit to jarjar123/correios that referenced this issue Dec 16, 2014
jarjar123 pushed a commit to jarjar123/correios that referenced this issue Dec 16, 2014
jarjar123 pushed a commit to jarjar123/correios that referenced this issue Dec 16, 2014
jarjar123 pushed a commit to jarjar123/correios that referenced this issue Dec 16, 2014
jarjar123 pushed a commit to jarjar123/correios that referenced this issue Dec 16, 2014
@pedro-teixeira pedro-teixeira added this to the v4.4.0 milestone Feb 27, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants