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

Ordenar pacotes cadastrados #10

Closed
igorsantos07 opened this issue Aug 27, 2017 · 16 comments
Closed

Ordenar pacotes cadastrados #10

igorsantos07 opened this issue Aug 27, 2017 · 16 comments

Comments

@igorsantos07
Copy link
Contributor

Seria legal poder indicar uma ordenação dos pacotes, e ver mais detalhes na listagem do /pacotes:

  • ordenar por pacote mais antigo, e daí, exibir tbm o número de dias desde a entrada do pacote no sistema dos Correios (ex: - PNXXXXXXXXXNL Minha encomenda (10 dias))
  • ordem alfabética. Eu estou usando tags ([DX] bugigangas) pra identificar a origem dos meus pacotes, e seria legal poder vê-los ordenados :)
@GabrielRF
Copy link
Owner

Se não me engano, ele ordena do que há mais tempo recebeu atualização para o que recebeu atualização mais recentemente, não?

@GabrielRF
Copy link
Owner

Essa issue vai dar um certo trabalho pra fechar...
Teria que mudar todo o esquema do select
Arquivo: rastreiobot.py
Linhas: 61

@igorsantos07
Copy link
Contributor Author

igorsantos07 commented Aug 28, 2017 via email

@andersonberg
Copy link
Contributor

A ideia seria um novo comando, por exemplo: /pacotes_ordena e/ou /pacotes_alfa? E a ordenação dos mais antigos seria pela data de entrada do pacote no sistema dos correios ou pela data de inserção no banco do bot?

@rougeth
Copy link
Collaborator

rougeth commented Sep 28, 2017

Acredito que a ordenação deveria ser pela entrada do pacote nos correios, pra quem usa o bot, a data de inserção no banco do rastreiobot não faz diferença.

@andersonberg
Copy link
Contributor

Beleza, comecei a trabalhar nessa issue e devo criar um PR em breve, daí vocês podem avaliar. Obrigado!

@rougeth
Copy link
Collaborator

rougeth commented Sep 29, 2017

@andersonberg se vc esperar até domingo pra mandar o PR, dá pra participar do hacktoberfest :D

@andersonberg
Copy link
Contributor

Ótimo, farei isso! :D

@andersonberg
Copy link
Contributor

Guys, eu criei um PR pra essa issue, se puderem revisar e testar #15

@andersonberg
Copy link
Contributor

Aceito críticas, sugestões, apelos... :D:D

@GabrielRF
Copy link
Owner

Ficou complexo pra mim..
@rougeth me ajuda ae!

@rougeth
Copy link
Collaborator

rougeth commented Oct 4, 2017

Vou tentar ver hoje a noite :D

@andersonberg
Copy link
Contributor

Se quiserem posso adicionar mais comentários no código explicando a lógica. E podem ir fazendo perguntas, sem problemas.

@rougeth
Copy link
Collaborator

rougeth commented Oct 4, 2017

Comentários é bom, principalmente nas partes menos triviais.

@andersonberg
Copy link
Contributor

Pronto, atualizei o PR com os comentários

@GabrielRF
Copy link
Owner

GabrielRF commented Nov 3, 2017

4ae0654
Ordenação alfabética ok.

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

No branches or pull requests

4 participants