-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add service orders patch #104
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fiz alguns comentários a respeito de classes não utilizadas e outras partem que podem ser melhoradas. Nada que impeça o merge.
@@ -0,0 +1,6 @@ | |||
class ServiceOrdersRequest(): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Essa classe está sendo utilizada na aplicação? Se sim, o método errors
é realmente necessário? Aparentemente ele só está chamando o pass
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Toda a pasta dtos foi feita pela @sarahraqueld , mas não sei onde nem mesmo se é usada. Então queria deixar nas mãos dela a deleção ou não.
@@ -0,0 +1,13 @@ | |||
class ServiceOrderRequest(): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Essa classe é necessária? Aparentemente ela não está sendo utilizada.
Responsáveis: @Adrilene @denisousa @sarahraqueld
Tipo: Feature
issue: #91
Descrição
Implementação da rota PATCH com os testes. Além disso, conserto dos testes anteriores, da ordem de serviço, que estavam com erros.
Schema refatorado com adição de um SchemaBase, com o objetivo de ser útil para as outras collections.
Checklist
Observações
O formato das pastas está diferente do formato da collection de itens.