-
Notifications
You must be signed in to change notification settings - Fork 1
PR/39 Solucion de colision de enumerados #40
Conversation
ocial/models.py
Outdated
class Category(Enum): | ||
SPORTS = 0 # , ("Sports") | ||
MUSIC = 1 # , ("Music") | ||
MARKETS = 2 # , ("Markets") | ||
RELAX_ACTIVITIES = 3 # , ("Relax activities") | ||
LIVE_CONCERT = 4 # , ("Live concert") |
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.
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.
Con tus cambios funciona perfe el CategoryEnum
. Sin embargo, otro enum que ahora mismo está afectado por lo mismo es el de TypeClient
(que no lo encuentro por ninguna parte en este PR, asi que supongo que hay que editar otros archivos, en swagger sale tmb).
SPORTS = "Sports" # , ("Sports") | ||
MUSIC = "Music" # , ("Music") | ||
MARKETS = "Markets" # , ("Markets") | ||
RELAX_ACTIVITIES = "Relax activities" # , ("Relax activities") | ||
LIVE_CONCERT = "Live concert" # , ("Live concert") |
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.
SPORTS = "Sports" # , ("Sports") | |
MUSIC = "Music" # , ("Music") | |
MARKETS = "Markets" # , ("Markets") | |
RELAX_ACTIVITIES = "Relax activities" # , ("Relax activities") | |
LIVE_CONCERT = "Live concert" # , ("Live concert") | |
SPORTS = "Sports" | |
MUSIC = "Music" | |
MARKETS = "Markets" | |
RELAX_ACTIVITIES = "Relax activities" | |
LIVE_CONCERT = "Live concert" |
.github/workflows/openapi.yml
Outdated
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.
Este commit tmb habría que droppearlo
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.
dropeado
|
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.
@adrrf @Domi-ATLAS En lo que al cliente generado respecta, ya todo bien (al menos de momento).
En mi opinión faltan por eliminar los comentarios que aún quedan en el CategoryEnum
por ser redundantes y no aportar nada de información extra, pero eso ya lo dejo en vuestras manos.
Generación API correcta, falta eliminación de los comentarios para mi gusto, pero no es un blocker para mí
No description provided.