Alteração do usuário associado ao carrinho. #456
Conversation
Associado um usuário existente ao carrinho
|
|
There was a problem hiding this comment.
Oi @ecureuill, tudo bem? Agradeço muito pela sua contribuição, com certeza vai ajudar a todos que usam o ServeRest para estudar, já que uma documentação clara e base de dados sanitizada é importante. 🥳
Gostei principalmente de ter identificado a causa do problema (preciso pensar em como evitar que esse erro volte a ocorrer, que tipo de teste faz sentido e se faz sentido investir tempo nisso).
Falta pouco para poder aprovar o pull request e publicar a sua alteração. Para seguir adiante vai ser preciso que faça 2 pequenos ajustes:
1. Ajustar a mensagem de commit para seguir o padrão de projeto
A mensagem de commit precisa seguir as regras do conventional commit, caso não conheça sobre tem alguns links nesse material do ServeRest sobre como contribuir no projeto: https://github.com/ServeRest/ServeRest/blob/trunk/.github/CONTRIBUTING.md#-commit
Mas caso queira ir por um lado mais eficiente, você pode executar o comando npm run commit no terminal para ter uma ajuda interativa na criação do texto de commit.
2. Fazer o mesmo ajuste na documentação
Identifiquei que no swagger.json há o mesmo problema de _id desatualizado, é interessante atualizar ele também.
Caso tenha qualquer dúvida, não hesite em pedir ajuda aqui no PR, novamente: Agradeço pela ajuda ao ServeRest.
Sobre as falhas de pipeline
É esperado que a pipeline quebre, pois os secrets do repositório não são acessíveis em repositório de fork, mas executei todos os testes localmente e todos passaram com sucesso 🙌🏼
Associado um usuário existente ao carrinho padrão
Description
Atualmente não é possível excluir os produtos padrões pois estão associados a um carrinho de um usuário inexistente.
Isto ocorreu no commit 3256274
How can the user experience this change?
Autenticado com o usuário padrão (
fulano@qa.com), será possível cancelar ou concluir o carrinhoaFOUqntef4iaOwWfg.Consequentemente, será possível excluir os produtos associados a este carrinho.
Documentation
Related Issues
PR Tasks