Verificar query e retorno de Services_GET

paulojna commented 2 years ago

GET /applications/services

[x] query a filtrar a collection pelos valores procurados
[x] schema a garantir exclusão mútua
[x] validação do formato UUID no serInstanceId
[x] Tratamento de erros e envio de uma instância ProblemDetails

jnbastos commented 1 year ago

Alterações feitas em: models.py

class ServiceGet schemas.py
service_get_schema services_controller.py
class ServicesController: utils.py
def none_to_empty_brackets
def camel_to_snake

commits: https://github.com/ATNoG/netedge-mep/compare/main...UMinho-Netedge:netedge-mep-uminho:serv_get

nflori commented 1 year ago

Olá João,

Qual o link onde estão as mais recentes versões do abaixo?

models.py

schemas.py services_controller.py utils.py

Obrigado.

Nicolás

De: João Bastos @.***> Enviado: 12 de julho de 2022 16:45:58 Para: UMinho-Netedge/netedge-mep-uminho Cc: Nicolas Francisco Lori; Assign Assunto: Re: [UMinho-Netedge/netedge-mep-uminho] Verificar query e retorno de Services_GET (Issue #6)

Alterações feitas em: models.py schemas.py services_controller.py utils.py

— Reply to this email directly, view it on GitHubhttps://github.com/UMinho-Netedge/netedge-mep-uminho/issues/6#issuecomment-1181931615, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AVRCTTXAWUFVR3UFUBLMXCLVTWHLNANCNFSM5Z6OBD7Q. You are receiving this because you were assigned.Message ID: @.***>

jnbastos commented 1 year ago

Olá Nicolás,

Segue o link da branch onde fiz as alterações:

https://github.com/UMinho-Netedge/netedge-mep-uminho/tree/serv_get

Para revisão aconselho a utilização da ferramento GitKraken (https://www.gitkraken.com/), que permite ver com muito maior facilidade todas as alterações feitas no código nas diferentes branches, permitindo ter uma vista mais geral do código também.

João

nflori commented 1 year ago

Olá João,

Podemos ter uma reunião os dois para falarmos um pouco do kraken.

Já fiz o download do Kraken e liguei-o ao github, mas não percebo como obtenho a imagem que me mostraste.

Se podemos reunir, dá-me um link e eu entro no link para partilharmos o ecran.

Obrigado.

Nicolas

De: João Bastos @.***> Enviado: 14 de julho de 2022 13:53:52 Para: UMinho-Netedge/netedge-mep-uminho Cc: Nicolas Francisco Lori; Assign Assunto: Re: [UMinho-Netedge/netedge-mep-uminho] Verificar query e retorno de Services_GET (Issue #6)

Olá Nicolás,

Segue o link da branch onde fiz as alterações:

https://github.com/UMinho-Netedge/netedge-mep-uminho/tree/serv_get

Para revisão aconselho a utilização da ferramento GitKraken (https://www.gitkraken.com/), que permite ver com muito maior facilidade todas as alterações feitas no código nas diferentes branches, permitindo ter uma vista mais geral do código também.

João

— Reply to this email directly, view it on GitHubhttps://github.com/UMinho-Netedge/netedge-mep-uminho/issues/6#issuecomment-1184412885, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AVRCTTTA5XDYDGXXGD62MH3VUAEWBANCNFSM5Z6OBD7Q. You are receiving this because you were assigned.Message ID: @.***>

jnbastos commented 1 year ago

Claro. Segue o link do zoom: https://videoconf-colibri.zoom.us/j/87005435677

nflori commented 1 year ago

Perguntas de Nicolas e respostas de João Bastos: 1) acerca do models.py: como consegues passar de 2 blocos que definem o dict para apenas um bloco? As linhas 821 a 835 form removidas porque ainda estava em fase de testes e estava a tentar perceber como funciona o mecanismo. Mas depois vi que o melhor seria criar a função interna bool_conv. Na remoção da 809 a 819 que depois virou em adição das 813 a 821 se reparar a única alteração foi mesmo de identação pois acrescentei mais um tab,

2) acerca do schemas.py: porque é que os "not required" mudaram? Porque não estavam a garantir a exclusão mútua do ser_instance_id, ser_category_id e ser_name, então alterei para quando fizer validate ele recorra ao schema e ele lance um erro caso não haja a tal exclusão mútua.

nflori commented 1 year ago

Declaração de João Bastos: Já corrigi a validação do formato UUID para o caso em que se passa uma lista de ser_instance_ids. No gitKraken pode se ver o update com a descrição "Update services_get: validate UUID format for multiple ser_instance_id" onde se encontram modificações em dois ficheiros mas a única que interessa é do services_controller.py.

Resposta de Nicolas Lori: Vi e pareceu-me bem.

UMinho-Netedge / netedge-mep-uminho

Verificar query e retorno de Services_GET #6