fix: Añadir información sobre las citas en la API#402
fix: Añadir información sobre las citas en la API#402sdelquin merged 4 commits intopythoncanarias:mainfrom
Conversation
|
Hola @sdelquin and @euribates, como vieron el draft ? Hay que hacer algún ajuste en los nombres? De esta forma puedo enviarles el PR y seguir con el otro issue, mil gracias! |
| path('v1/events/all/', views.all_events, name='all_events'), | ||
| path('v1/events/', views.active_events, name='active_events'), | ||
| # Quotes | ||
| path('v1/quotes/', views.quote, name='quote'), |
There was a problem hiding this comment.
Preferiria llamar al método random_quote, asi creo que quedaría más claro
| return [serializer_sponsor(sponsor) for sponsor in sponsors] | ||
|
|
||
| @api | ||
| def quote(request): |
There was a problem hiding this comment.
Preferiria llamar al método random_quote, asi creo que quedaría más claro
El serializer no tiene que saber si la cita ha sido elegida al azar o no, le es indiferente
|
Señores disculpas por la confusión, regreso a mi usuario original @navascogt con los comentarios que han adicionado por acá. |
|
@sdelquin @euribates gracias por aceptar los cambios y hacer los cambio sugeridos, entiendo que con este issue ya no tengo que hacer nada verdad? Y por otro lado ya puedo regresar al issue original #378 que fue el que me trajo a este issue, por allá les dejo mis comentarios. |
sdelquin
added
hacktoberfest-accepted
4 participants
No description provided.