j-alves-c / trabalho-gcs

Repositório do Trabalho para a Disciplina de Gerência de Configuração de Software 2022-1
1 stars 0 forks source link

Apagar campos não utilizados na adição de sapatos #22

Closed melonathalia closed 2 years ago

melonathalia commented 2 years ago

Quando adicionamos campos a mais não existe nenhum tratamento para esse caso, seria legal que esses campos extras fossem excluídos na hora de confirmar a venda. @j-alves-c poderia dar uma olhada nessa parte?

j-alves-c commented 2 years ago

Nath, você poderia mostrar melhor essa ideia? Algum vídeo de como isso ocorre?

guilhermectprado commented 2 years ago

Possui alguma imagem ou vídeo pra facilitar o entendimento? Ou simplesmente melhorar a descrição?

melonathalia commented 2 years ago

Segue um exemplo de como fica os campos quando remove algum sapato:

https://user-images.githubusercontent.com/38357516/175786177-e793f7b9-cece-4783-bb48-c3b4d97498b4.mp4

Então minha ideia é que esse campo fosse removido antes de finalizar a venda.

j-alves-c commented 2 years ago

Entendi, mexo sim!

j-alves-c commented 2 years ago

A funcionalidade já existia em si, o apagamento de campos, mas não durante a venda corrente. Apenas eram apagados todos os campos quando se finalizava a venda. Dessa forma, o commit 5a03d588c033e5fc6e7c6c7361cc0b6691c63198 se faz uma atualização de performance e não em si de uma funcionalidade. Assim, o campo extra é apagado quando se remove o sapato da vendda.