[WIP] 7 - Implementar HTTP DELETE/plants/{id} - #21
javierlopezdeancos wants to merge 15 commits into
Conversation
javierlopezdeancos
changed the title
mdelapenya
left a comment
mdelapenya
left a comment
There was a problem hiding this comment.
De momento tiene buena pinta, quiero bajarlo y probar. No obstante te dejo algunas cosillas que he visto
| } | ||
|
|
||
| // SearchQueryByIndexNameResult wraps a search result | ||
| type SearchQueryByIndexNameResult map[string]interface{} |
There was a problem hiding this comment.
El index es algo que se pone en el cliente, yo lo llamaría SearchQueryResult
| } | ||
|
|
||
| // SearchQueryByIndexName struct | ||
| type SearchQueryByIndexName struct { |
There was a problem hiding this comment.
El index es algo que se pone en el cliente, yo lo llamaría SearchQuery
There was a problem hiding this comment.
eliminarias ese IndexName string del struct entonces?
| @@ -0,0 +1,9 @@ | |||
| package plants | |||
There was a problem hiding this comment.
Yo dejaría ésto como un método en el main.go, para no tener un paquete con sólo un método público, o al menos en un fichero dentro del paquete raíz
There was a problem hiding this comment.
Bueno, yo también he tenido esa duda, al final, desde mi opinión inexperta, lo hice así para tener un dominio que no dependa de ninguna implementación, en este caso, nuestro dominio son las planticas y su gestión, en el main tenemos el problema que tenemos un buen salpicado de marisco de implementación del servidor, con gin, elastic logs tracking and monitoring
There was a problem hiding this comment.
la idea era tener un paquete plants de nuestra entidad de dominio y aquí empiezan mis dudas
There was a problem hiding this comment.
según veo y a efectos practicos, un cliente lo unico que debería pedir es un search('plants'),
ese plants se lo daría el contexto de Gin
There was a problem hiding this comment.
crearía un api.go en el raíz, y allí todos estos metodicos
| @@ -0,0 +1,11 @@ | |||
| package plants | |||
There was a problem hiding this comment.
Lo mismo que para el Delete, lo dejaría en el main.go, o al menos en un fichero dentro del paquete raíz
2 participants
#7
Esta PR es un DRAFT para revisión
Closes #7