Skip to content
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

Merged
merged 6 commits into from
Jul 2, 2020
Merged

Conversation

sarahraqueld
Copy link

@sarahraqueld sarahraqueld commented Jun 29, 2020


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

  • Testes foram implementados (novos ou não)
  • Issue foi definida no PR (Linked Issue)
  • Pessoas contribuidoras foram definidas no PR (Assigners)

Observações

O formato das pastas está diferente do formato da collection de itens.

@sarahraqueld sarahraqueld marked this pull request as draft June 29, 2020 12:50
@Adrilene Adrilene requested review from Edely and juliakastrup June 30, 2020 14:24
@Adrilene Adrilene linked an issue Jun 30, 2020 that may be closed by this pull request
4 tasks
@Adrilene Adrilene marked this pull request as ready for review June 30, 2020 14:25
Copy link

@Edely Edely left a 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():
Copy link

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.

Copy link

@Adrilene Adrilene Jul 2, 2020

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.

api/v2/controllers/service_orders_many_controller.py Outdated Show resolved Hide resolved
@@ -0,0 +1,13 @@
class ServiceOrderRequest():
Copy link

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.

api/v2/utils/util_update.py Outdated Show resolved Hide resolved
@oxechicao oxechicao merged commit c9bddec into develop Jul 2, 2020
@oxechicao oxechicao deleted the feature/91_new_patch_service_order branch July 2, 2020 15:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Recriar rota PATCH para cadastro da Ordem de Serviço
6 participants