pedroaraujo20
commented
4 years ago Boa noite Cássio, Feliz Natal!!
De: Cássio Santos notifications@github.com Enviado: quarta-feira, 25 de dezembro de 2019 14:14 Para: pedroaraujo20/favsgithub favsgithub@noreply.github.com Cc: Subscribed subscribed@noreply.github.com Assunto: [pedroaraujo20/favsgithub] Feedback - Skilopay (#1)
Boa tarde Pedro, Feliz Natal!
Segue um breve Feedback sobre seu desafio, considere tudo como sugestões, se discordar de algo ou ficar com alguma dúvida estou disponível para discutirmos sobre.
Review Geral:
- Segundo os requisitos, deveria ser possível remover um item da lista de favoritos ao clicar novamente no botão de favoritar em um item na tela de Trendings (Permitir remover dos favoritos)
- Você poderia ter utilizado typescript para validação de tipos e para melhorar o intellisense durante o desenvolvimento ou utilizar o Proptypes se realmente fosse de preferência utilizar javascript
- Na linha 46 da página Trendings existe um acesso perigoso a um campo de uma variável que é possivelmente undefined [Captura de Tela 2019-12-25 às 12 52 42]https://user-images.githubusercontent.com/7540050/71448043-7e00ce80-2715-11ea-80ba-ab09550c1aef.png
- Esqueceu de usar SafeAreaView na tela Repository. Como usou nas outras, deve ter apenas se esquecido. [Captura de Tela 2019-12-25 às 12 55 51]https://user-images.githubusercontent.com/7540050/71448085-f9fb1680-2715-11ea-94e3-a1b1d172cc3d.png
- Bom Readme, com preview do aplicativo e tutorial de instalação. Senti falta apenas do preview do app no iOS.
- Boa organização de código em geral.
- Um arquivo de configurações para manter as cores padrões do app facilitaria o desenvolvimento e possíveis alterações de design sem ter que alterar cada arquivo ou tentar lembrar onde deve alterar.
- Cuidado com utilização de objetos vindos de serviços de terceiros, por exemplo, no componente TrendingList você passa toda uma prop chamada data que possui a mesma estrutura da resposta da Api, trabalhando com testes fica muito difícil mockar uma estrutura dessa ou reaproveitar o componente, é preferível passar 2 props (name e totalCount) e fazer com que o componente pai fique responsável por desestruturar esse objeto (Adicione default props para evitar erros inesperados). (Isso facilita quando estiver trabalhando em Layouts e o serviço de terceiros estiver offline, por exemplo)
- Poderia ter trabalhado um pouco mais nos testes da aplicação, ao menos testes de snapshots em cada um dos componentes com props variadas.
- Na tela de Trendings, poderia ter utilizado o loadMore do FlatList e o fetchMore do useQuery para pegar mais itens dos trendings do github.
- Mal uso de reducers do Redux. Os reducers devem ser utilizados para controlar exclusivamente o estado da aplicação. Alguns de seus reducers apresentam feedback visual como um side effect. (O uso do immer foi interessante)
- Commits em inglês e Readme em português, não é um grande problema, mas é preferível manter consistência.
Diferenciais
- Utilizar Apollo-client
- Criar issues e resolvê-las a partir de pull requests
- Documentação no código e repositório
- Arquivos pequenos com poucas responsabilidades
- Estado da aplicação bem estruturado
- Usabilidade e feedback para o usuário no carregamento da consulta
- Seguir algum Javascript Style Guide
- Utilizar alguma ferramenta de CI
- Testes unitários
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://github.com/pedroaraujo20/favsgithub/issues/1?email_source=notifications&email_token=AEIPZYXNBGSLEJFXSE3TG3DQ2OBFPA5CNFSM4J7FTXG2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4ICTZF2A, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AEIPZYRSKLMHX6J7ZLUI4FDQ2OBFPANCNFSM4J7FTXGQ.
ssscassio
Boa tarde Pedro, Feliz Natal!
Segue um breve Feedback sobre seu desafio, considere tudo como sugestões, se discordar de algo ou ficar com alguma dúvida estou disponível para discutirmos sobre.
Review Geral:
typescriptpara validação de tipos e para melhorar o intellisense durante o desenvolvimento ou utilizar o Proptypes se realmente fosse de preferência utilizar javascriptnameetotalCount) e fazer com que o componente pai fique responsável por desestruturar esse objeto (Adicione default props para evitar erros inesperados). (Isso facilita quando estiver trabalhando em Layouts e o serviço de terceiros estiver offline, por exemplo)Diferenciais