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

Challenge #1

Closed
wants to merge 58 commits into from
Closed

Challenge #1

wants to merge 58 commits into from

Conversation

leandrooriente
Copy link

@leandrooriente leandrooriente commented Aug 8, 2016

Algumas observações:

  • O projeto possui poucas dependências globais. Node, NPM, Webpack e o http-server que uso apenas para simular uma API com os valores dos imóveis. De qualquer forma coloquei um fallback local pro http-server apesar de ser recomendada a instalação global.
  • Não pude trabalhar durante essa semana e participei do Front in BH no sábado o que limitou bastante meus horários. Só consegui trabalhar no sábado depois do evento e no domingo. https://github.com/leandrooriente/desafioimoveis/graphs/commit-activity
  • Ainda não achei uma ferramenta consistente de testes para React + Redux. Estou cogitando Enzyme ou o Jest mesmo. Pretendo estudar isso essa semana.
  • A lib sugerida para o cálculo do valor do financiamento é projetada para calcular rendimentos de aplicações. Optei por usar a fórmula do Banco Central do Brasil
  • Comecei o projeto com Mobx, mas optei pelo Redux no decorrer dele. Fiquei meio receoso com as mágicas do Mobx e com o fato deles sugerirem que você conecte em todos os componentes. Achei o fluxo unidirecional do Redux mais seguro ao longo prazo apesar de mais verboso.
  • Optei por usar stateless components sempre que possível. Só precisei lidar com o lifecycle do React em um componente. Aparentemente a equipe do React vai melhorar a performance deles num futuro breve e o código fica mais legível IMHO.
  • Utilizei material design para experimentar um framework diferente mas tive algumas dificuldades. O projeto ainda está muito imaturo. Tem 4 warnings do React por conta de um componente de slider. Já está corrigido mas ainda não foi publicado no NPM. Além disso é necessário adicionar uma lib para lidar com o evento de tap nativo do IOS.

@leandrooriente
Copy link
Author

@demetriusnunes sobre testes estou cada vez mais convencido de que possuem um custo benefício ruim. Dan Abramov já falou sobre isso recentemente https://twitter.com/dan_abramov/status/757316256861151233

Acho bem desnecessário para componentes React. Para as actions e o estado esperado no caso do Redux acho importante, mas para o componente em si acho meio meh.

O próprio PropType se configurado corretamente te alerta se algo deveria chegar de um jeito e chegou de outro. Gosto bastante de ser bem certinho nesse quesito e validar campo por campo cada componente.

Não sei se é aplicável num ambiente de startup que muda a interface a cada 3 horas.

@leandrooriente
Copy link
Author

Dei uma arrumada na interface.

  • Coloquei umas fotos dinâmicas de fundo com a API do 500px.
  • Deixei a experiência melhor no mobile

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.

None yet

1 participant