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

Adicionada compatibilidade com pacote de produtos. #96

Merged
merged 1 commit into from Jul 20, 2015

Conversation

Projects
None yet
3 participants
@jarjar123
Contributor

jarjar123 commented Jul 15, 2015

Closes #97

  • Adicionada compatibilidade com pacote de produtos.
  • Registrada exibição de área de risco e número de volumes ao título do frete.
  • Refatorados alguns métodos para facilitar estas alterações.

@pedro-teixeira pedro-teixeira added this to the v4.6.0 milestone Jul 15, 2015

@pedro-teixeira pedro-teixeira self-assigned this Jul 15, 2015

$successCode = '0';
$hasValidQuote = array_key_exists($successCode, $errorList);
if (!$hasValidQuote) {
$displayErrorList = array('-3', '-6', '-10', '-33', '-888', '7', '99');

This comment has been minimized.

@pedro-teixeira

pedro-teixeira Jul 20, 2015

Owner

Não seria interessante passar esses dados para o config? Como o soft_errors?

This comment has been minimized.

@jarjar123

jarjar123 Jul 20, 2015

Contributor

Os Correios definem uma cotação de sucesso como Erro 0. Mas infelizmente ainda há 3 outros casos, onde a cotação também é válida Erros 009, 010 e 011. São cotações com erro; porém, com preço, prazo e algum alerta adicional. Foi o que configurei como soft_errors.

Os outros erros não possuem preço e prazo (não são soft_errors). Então acho que seria melhor adicioná-los a uma outra variável.

@jarjar123 jarjar123 force-pushed the jarjar123:bundle_product_and_more branch from d17a62a to 3e16026 Jul 20, 2015

pedro-teixeira added a commit that referenced this pull request Jul 20, 2015

Merge pull request #96 from jarjar123/bundle_product_and_more
Adicionada compatibilidade com pacote de produtos.

@pedro-teixeira pedro-teixeira merged commit dde9328 into pedro-teixeira:master Jul 20, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@jarjar123 jarjar123 deleted the jarjar123:bundle_product_and_more branch Jul 21, 2015

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