Feat/edicao horas - Githubissues

m-dantas commented 4 years ago

Refere-se a issue https://github.com/SPURB/apropriacaohoras/issues/52

Adicionado edição de horas
Validações de registro
Exibição de mensagem pelo modal

netlify[bot] commented 4 years ago

Deploy preview for apropriacaohoras ready!

Built with commit 290d48f96ef6339ed002d8f1dd445fbc8b544579

https://deploy-preview-63--apropriacaohoras.netlify.app

TravisBuddy commented 4 years ago

Hey @m-dantas,
Your changes look good to me!

View build log

TravisBuddy Request Identifier: f34f4e10-af40-11ea-aad4-a340821c2822

yubathom commented 4 years ago

Pendências e alterações:

[x] Incluir o botão "incluir atividade" para que usuário possa incluir um item com os campos vazios
[x] Horas extras devem estar desabilitadas até que a soma das horas do dia esteja com 8 horas
[x] Registrar horas deve voltar para /registrar
[x] Finalizado o fluxo de atualização, o botão do modal de sucesso "Voltar" precisa voltar para /registrar

Bugs (relacionadas à esta pr ou que não constam na versão da master):

[x] Muitas vezes a soma de horas não bate.

O esperado é que a soma de horas dos itens do formulário seja 4horas e não 8h.

Screenshot_2020-06-15 SPUrbanismo Apropriação de horas(2)

Aqui tem dois erros. Extrapolou o número de horas e a soma está incorreta.

[x] Itens da lista "Horas já registradas" ficam duplicados após a atualização em /editar
[x] Itens da lista "Horas já registradas" não correspondem ao dia selecionado

aqui apenas o dia 15 foi selecionado
[x] Scroll travado por alteração no overflow do body. Ao entrar na seção e ir para outra o overflow da body está sendo alterado.

Considerações:

[x] Max-width da seção deve ser o valor da variável $desktop (1200px)
[x] Antecipar breakpoint para deixar o conteúdo do select sempre visível

@m-dantas, fiz um pente fino da perspectiva do usuário do site. Veja o que rola de arrumar sem aumentar demais a complexidade. Se achar que está ficando uma bola de neve deixa quieto e vai para o próximo item. Em seguida, o que ficar pendente vou contribuir nesta pr. De resto achei que ficou bem bom para uma feature tão complexa. Vamos fazer estes ajustes pra deixar 100%.

os testes estão ok:

     Spec                                              Tests  Passing  Failing  Pending  Skipped
  ┌────────────────────────────────────────────────────────────────────────────────────────────────┐
  │ √  admin\card-router.spec.js                00:12        3        3        -        -        - │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ √  admin\equipes.spec.js                    00:16        3        3        -        -        - │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ √  admin\fase.spec.js                       00:27        5        5        -        -        - │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ √  admin\projetos.spec.js                   00:23        5        5        -        -        - │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ √  admin\relatorio-detalhado.spec.js        00:10        2        2        -        -        - │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ √  admin\relatorios.spec.js                 00:04        1        1        -        -        - │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ √  admin\subatividades.spec.js              00:25        5        5        -        -        - │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ √  login\login.spec.js                      00:04        4        4        -        -        - │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ √  registrar-horas\registrar.spec.js        00:33        7        7        -        -        - │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ √  registrar-horas\ui-calendario.spec.      00:32        4        4        -        -        - │
  │    js                                                                                          │
  └────────────────────────────────────────────────────────────────────────────────────────────────┘
    √  All specs passed!                        03:10       39       39        -        -        -

m-dantas commented 4 years ago

Beleza @yubathom vou dar uma atenção nesses problemas porém tem uns que eu até estranhei tipo os do break points width: 1200 Screenshot_16

width: 874

E sobre os numeros de horas extrapolado e coloquei um disabled e uma mensagem no botão de salvar, como ilustra a imagem 1 mas vou adicionar os disabled de add horas extras tbm.

E sobre as horas não mostradas corretamente e as data duplicadas já tenho uma ideia de como resolver, acredito que seja por causa do vuex, acho que faltou eu colocar para zerar o state.

Enfim qualquer dificuldade eu falo para ti e obrigado pelo feedaback mano.

yubathom commented 4 years ago

@m-dantas, a revisão foi feita neste link aqui: https://deploy-preview-63--apropriacaohoras.netlify.app/

é um build (npm run generate) do último commit desta pr

m-dantas commented 4 years ago

@yubathom foi desse mesmo que testei as breakpoint do css mas vou revisar se algum css precisa colocar alguma prop em mozilla

TravisBuddy commented 4 years ago

Hey @m-dantas,
Your changes look good to me!

View build log