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

Fix/show all GitHub contributors #353

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

hugobbi
Copy link
Contributor

@hugobbi hugobbi commented Jun 22, 2024

Notei um pequeno problema na sessão 'Sobre nós' do site. O código que busca os contribuidores do github (tanto do back como do front) chama a api do github para buscar os contribuidores dos repositórios. O problema é que, da forma como está, a api devolve apenas os 30 contribuidores mais recentes, dado que os contribuidores ficam divididos em páginas, e o limite padrão é de 30 entradas por página. Assim, minha mudança foi pequena: primeiro eu aumentei o limite de entradas por página para 100 e então fiz um código para, se há outra página disponível, ele busca por mais contribuidores nessa página. Dessa forma, todo mundo que fez alguma contribuição irá aparecer na página, não importa quantos forem futuramente. Eu testei essa funcionalidade e parece estar funcionando normalmente, agora aparecem todos os contribuidores mesmo.

Antes:
Captura de tela de 2024-06-22 14-38-32

Depois:
Captura de tela de 2024-06-22 14-38-45

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

Successfully merging this pull request may close these issues.

1 participant