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

Unificando listagens de escola #51

Merged
merged 2 commits into from
Oct 2, 2018

Conversation

victorsenam
Copy link
Contributor

@victorsenam victorsenam commented Sep 28, 2018

Criando os componentes SchoolList e School para possibilitar a reutilização de código entre os containers Register e Results. Isso contribui para as issues #10 e #11, deixa o código mais fácil de ler (na minha opinião) e mais fácil de atualizar.
Tem algumas coisas polêmicas nesse PR eu posso tentar arrumar se acharem ruim:

  1. Eu errei na hora de commitar, eu ia criar um commit que só cria estes componentes e usa em Results para depois criar um outro commit que os torna usáveis (e usa) em Register, mas eu acabei fazendo tudo no mesmo e a mensagem de commit, inclusive, ficou esquisita. Se acham que isso é um problema posso ajustar.
  2. Da forma que fiz, o componente School só mostra a informação que recebe, isto é, se ele não recebe algum campo de informação sobre a escola, ele ignora. É assim que eu diferencio a informação que é mostrada entre os dois containers, eu simplesmente não passo as informações que não quero que um ou outro mostre. Fazer isso desse jeito pode ser um pouco confuso para quem for ler o código depois, mas acho que tá tudo bem.
  3. O posicionamento do ícone na página de registro mudou um pouco, foi disso
    screenshot from 2018-09-28 00-12-44
    pra isso
    screenshot from 2018-09-28 00-12-48

Separa a lógica de paginação e a renderização da informação da escola em dois componentes.
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.

2 participants