search

FeliciLab / registrocovid-front

GNU General Public License v3.0
2 stars 0 forks source link

Hotfix/info gerais #80

Closed dayanyespindola closed 4 years ago

dayanyespindola commented 4 years ago
  1. Havia um problema que quando a página de informações gerais de um paciente era carregada os dados não eram exibidos.

Isso ocorria porque os dados do paciente estavam no contexto mas não eram trazidos do contexto para o componente GeneralInfo.js. Então, agora o estado inicial desse componente verifica se existe um paciente no contexto ou se um novo paciente está sendo cadastrado.

  1. Havia um outro problema: ao cadastrar um novo paciente, a pessoa é direcionada para para categorias e pode visualizar as informações gerais desse paciente. Porém elas não eram exibidas.

Isso acontecia porque a resposta do post que salva um novo paciente é colocada como paciente no contexto. Porém essa resposta não tem todos os dados que o form de Informações precisa. A solução por hora foi complementar esses dados com os que foram inputados no form.

PS:

  1. A gente tá com objetos Paciente diferentes entre backend e frontend (template e localstorage) e isso faz com que a transformação dos dados seja necessária em alguns lugares.

    • O que acham de criar uma pasta model e ter uma classe paciente?

  2. Na busca pelos dados de pacientes passando fields, os fields que não estão selecionados trazem valor null quando na verdade existe um valor para esse dado.

    • O que acham de não trazer os campos que não estão sendo selecionados?
ericsonmoreira commented 4 years ago

Sobre os PS's

  1. O que acham de criar uma pasta model e ter uma classe paciente? Não vejo problemas quanto a isso.
  2. O que acham de não trazer os campos que não estão sendo selecionados? Concordo. Mas isso seria feito no front ou no back?
dayanyespindola commented 4 years ago

Sobre os PS's

  1. O que acham de criar uma pasta model e ter uma classe paciente? Não vejo problemas quanto a isso.
  2. O que acham de não trazer os campos que não estão sendo selecionados? Concordo. Mas isso seria feito no front ou no back?

@ericsonmoreira conforme conversamos no DeskCheck, vamos marcar uma conversa. entre fronts e backs depois que eu fizer essa refatoração de usar a classe Paciente no model.