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

Adicionar paginação na listagem #10

Open
hebit opened this issue Apr 18, 2021 · 4 comments
Open

Adicionar paginação na listagem #10

hebit opened this issue Apr 18, 2021 · 4 comments
Labels
good first issue Good for newcomers

Comments

@hebit
Copy link

hebit commented Apr 18, 2021

Olar,

O projeto no link de demo está com erro 500 (caso não tenham notado).
Baixei o projeto e vi que os dados não estão paginados, imagino que pra chamada a API seria um tanto custoso receber todos os dados de vez (considerando que são +10k de CID's), vocês pretendem implementar paginação?

@a21ns1g4ts
Copy link
Owner

@hebit Realmente o base ficou off por algum tempo, Obrigado por notificar, em breve vai ter um patrocínio para manter a infra escalada, mas por enquanto estamos assim 😆 . Antes de resolver algumas questões como o suporte para múltiplas versões do cid e sincronização com os dados diretamente da OMS o objetivo inicial era apenas prover 2 end-points simples com esses dados estáticos. Em breve vou avançar com essas ideias e provavelmente passar a usar o sem ver para organizar a compatibilidade ao longo da implementação de novos recursos. Até lá sinta-se livre para mandar uma PR com a paginação, As duas únicas observações que eu tenho a fazer é que seja mantida a compatibilidade com os end-points e também que seja implementada no repositório de arquivo a lógica da paginação, e também os documentos readme.md e index.blade.php.

@a21ns1g4ts a21ns1g4ts added the good first issue Good for newcomers label Apr 18, 2021
@a21ns1g4ts a21ns1g4ts changed the title Paginação Adicionar paginação na listagem Apr 18, 2021
@hebit
Copy link
Author

hebit commented Apr 21, 2021

@Atiladanvi Show! No momento estou bem ocupado mas se me sobrar um tempo tento fazer o PR 😄

@a21ns1g4ts
Copy link
Owner

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

3 participants
@a21ns1g4ts @hebit and others