search

ecomplus / app-pagseguro

E-Com Plus app to integrate PagSeguro
MIT License
0 stars 1 forks source link

Minor fixes para a rotina de atualização dos pedidos #8

Closed leomp12 closed 4 years ago

leomp12 commented 4 years ago

Me parece que o start estava sendo chamado duas vezes na inicialização, uma imediatamente e outra 10min depois, aparentemente isto também faria o processo rodar duas vezes a cada 12h (com 10min de intervalo), estou certo @talissonf ?

Limitei os resultados da lista de pedidos a 20, também passei a filtrar pela data de criação a partir de 7 dias atrás e inverti a ordenação (financial_status.updated_at ASC), ou seja, vamos buscar até 20 pedidos criados nos últimos 7 dias, começando pelos que foram atualizados a mais tempo (mais desatualizados), faz sentido?

leomp12 commented 4 years ago

Apesar de passar a limitar os resultador na lista de pedidos, acho que ainda precisamos de intervalos entre as requisições na API do PagSeguro, ainda corremos o risco de receber 503, não @talissonf ? A não ser que isto já esteja sendo feito pelo seu pagseguro-client...

talissonf commented 4 years ago

Apesar de passar a limitar os resultador na lista de pedidos, acho que ainda precisamos de intervalos entre as requisições na API do PagSeguro, ainda corremos o risco de receber 503, não @talissonf ? A não ser que isto já esteja sendo feito pelo seu pagseguro-client...

Nos meus testes não rolou 503, mesmo com um volume grande de pedidos. Não estou usando loop, estou chamando a função enquanto houve pedidos no array talvez seja por isso que os limites estão sendo respeitados.

talissonf commented 4 years ago

Vou da merge se houver algum problema com limite no pagseguro eu faço um retry no script