Feature US72-CheckList

[DanMke]

Descrição

Foi feita a feature da linha de cuidado, em que paciente e responsável tem acesso, de acordo com a idade do paciente e os procedimentos feitos, ele preencherá os checkboxes da tabela para manter um controle dos procedimentos que já foram feitos ou que deverão ser feitos. Se o paciente tiver menos de 13 anos, ele não poderá editar a linha de cuidado e o responsável deverá preencher. A tradução foi feita.

Issue Relacionada

#162 ## Como Isso Foi Testado? Foi testado por meio de testes unitários e testes manuais. ## Capturas de Tela     Mensagem sobre procedimentos não feitos (usuário paciente):  Mensagem sobre procedimentos não feitos (usuário responsável):  ## Tipos de Mudanças

• [ ] Bug fix (alteração que corrige uma issue e não altera funcionalidades já existentes)
• [x] Nova feature (alteração que adiciona uma funcionalidade e não altera funcionalidades já existentes)
• [ ] Alteração disruptiva (Breaking change) (Correção ou funcionalidade que causa alteração nas funcionalidades existentes)

Checklist:

• [x] Meu código segue o estilo de código desse projeto.
• [x] Meus commits seguem o padrão de estilo desse projeto.
• [ ] Minha alteração requer uma alteração na documentação.
• [ ] Eu atualizei a documentação de acordo.
• [x] Eu li o documento de Contribuição (CONTRIBUTING).
• [x] Eu adicionei testes para cobrir minhas mudanças.
• [x] Todos os testes novos e existentes passaram.

[guilherme1guy]

Fizemos alterações visuais nas páginas:

---

No final da tabela:

---

Imagem do usuário aparece na lista de pacientes:

[marimendes]

Primeiro o elogio porque eu adorei o frontend do checklist <3

[marimendes]

1 - Eu não testei ainda com outras imagens, mas com a imagem padrão fica estranho.

2- Não pode existir datas futuras: Ta menos um porque eu criei um usuário e coloquei o aniversário dele pra amanha.

3- Para nós que já sabemos da regra de negócio do site: só pode marcar no checklist no checklist se for acima de 13 anos e possuir um responsável... fica tranquilo. Mas para aqueles que não sabem, a mensagem fica muito genérica de erro fica muito genérica. Tipo, ta eu não posso marcar no checklist, mas por que não? Acho que deveria ser informado para usuário. Pode ser na mensagem de erro ou em um campo de testo em cima do checklist.

4 - Mesma coisa do pr de prontuário, vocês estão esquecendo do botão de voltar.

[marimendes]

Testei com outra imagem e realmente fica bem estranho. PODE, não sei se é isso, ser uma solução limitar o tamanho das imagens.

[guilherme1guy]

@marimendes sobre o tópico 2: creio não fazer parte dessa história. A idade que aparece é um calculo feito a partir da data de nascimento que existe no user.

E sobre o 1: na print que você mandou a imagem está achatada, é esse o problema? Se for, isso não ocorreu no meu dispositivo (tem até uma print da lista num comentário que fiz anteriormente).

[marimendes]

O tópico 2 vou colocar pra ser refatorado depois então e quanto a imagem, eu testei com outra e mesmo assim ficou achatada ou alongada.

[guilherme1guy]

O mesmo erro acontece no fórum com a mesma imagem? Ele também tem exibição de imagem do usuário.

[marimendes]

Não, no fórum acontece aquele outro bug:

[guilherme1guy]

Eu vou investigar isso assim que possível. Mas se for algo relacionado a limitar o tamanho da imagem do usuário, acredito que entra na mesma história do tópico 2 (da idade).

[marimendes]

@guilherme1guy olhei aqui no pc do sconeto e o dele também não ta bugado. Pode ignorar esse tópico então.

[guilherme1guy]

Ok

[sconetto]

Vamos lá galerinha do mal, as funcionalidades eu vi que a Mari apontou alguns bugs e eu já sugeri via slack algumas correções, agora os bugs que eu encontrei: Essa mensagem de verifique foi uma ideia bacana mas ela tá aparecendo em muitas páginas, mesmo quando você tá no checklist a mensagem tá lá, filtre a mostragem dela (colocar um timeout seria uma opção). Outro problema dessa mensagem é que ela tá fora de um container ai não tá condizendo com o frontend atual.

Um outro exemplo do que foi dito acima.

Eu não tive a oportunidade de verificar a fundo o código mas essas excessões tem que ter mensagens mais expressivas do que está acontecendo! Podemos olhar na próxima reunião juntos, mas se puderem já corrijam!

Aqui é um outro problema (apesar da mensagem que eu reclamei na primeira imagem tá aqui de novo), é que quando a gente tem mensagens de erro e mensagens de sucesso elas vão acumulando e descendo o conteúdo, a ideia aqui seria talvez colocar um timeout nessa mensagem, ou dar um refresh na página pra aparecer só uma vez a mensagem de sucesso a cada ação.

Enquanto aos problemas do codeclimate acredito que estão Ok, são negociáveis. O código eu vou olhar mais profundamente assim que eu analisar todos os prs e tiver um tempo para isso.

[guilherme1guy]

@marimendes @sconetto fiz algumas alterações. As mensagens de erro/sucesso agora somem de forma automática. A mensagem de erro na alteração da checklist ficou mais amigável/explicativa. Agora o usuário não logado é redirecionado para a tela de login. O aviso de procedimentos atrasados agora está em um container (obedece o layout).

@sconetto sobre a mensagem do atraso estar em muitas telas: na minha opinião é essa a ideia, por causa do teor da mensagem, é um aviso de que um procedimento importante pode estar atrasado. Eu adicionei um botão para fechá-la, mas ela deve reaparecer nas outras páginas (tirando a homepage).

[guilherme1guy]

Sobre a mudança na porcentagem de coverage: o code climate decidiu começar a analisar um arquivo gerado pelo cookie cutter (users/adapters.py), entretando na develop ela já tem cobertura.