richecr / CamaraDosDeputados

Aplicativo web da câmara dos deputados usando ReactJS.
https://quem-gasta-mais.netlify.app/
MIT License
43 stars 31 forks source link

Create deputados page #85

Closed tuliocll closed 3 years ago

tuliocll commented 3 years ago

77

Descrição do bug/feature: Criar tela de listagem dos deputados

Solução Criar novos componentes; Card de deputado, tag, paginação, loading e pesquisa.

Coisas a fazer Nada

ghost commented 3 years ago

Olá @tuliocll, eu baixei suas alterações aqui e executei, mas achei um problema na paginação das despesas, quando eu clico para passar para a próxima, nada acontece, ele não atualiza a página, pelo que vi ele não está fazendo a requisição a API.

Também está com um erro no build(o log não dá muitos detalhes, mas vou mandar um print abaixo do erro), antes do seu último commit estava com esse erro nos arquivos scss que importavam o componente Header(foi alterado recentemente) e os seus arquivos estavam apontando para o Main(nome antigo do componente), mas vi que você alterou os arquivos já. Mas acho que o erro ainda está nesses arquivos.

build_erro

tuliocll commented 3 years ago

Acabei de fazer outro commit, deve corrigir

tuliocll commented 3 years ago

Olá @tuliocll, eu baixei suas alterações aqui e executei, mas achei um problema na paginação das despesas, quando eu clico para passar para a próxima, nada acontece, ele não atualiza a página, pelo que vi ele não está fazendo a requisição a API.

Também está com um erro no build(o log não dá muitos detalhes, mas vou mandar um print abaixo do erro), antes do seu último commit estava com esse erro nos arquivos scss que importavam o componente Header(foi alterado recentemente) e os seus arquivos estavam apontando para o Main(nome antigo do componente), mas vi que você alterou os arquivos já. Mas acho que o erro ainda está nesses arquivos.

build_erro

Essa parte da paginação nas despesas foi por que eu remodelei o componente de paginação e não adaptei ele na tela das despesas, eu comentei no issue #75 , não quis mexer pra não atrapalhar quem pegou essa task.

ghost commented 3 years ago

Pronto, desculpa pela demora em analisar as alterações. Acho que está tudo correto agora. Vou aceitar e comentar na issue #75.