Skip to content
This repository has been archived by the owner on Jun 10, 2024. It is now read-only.

PR/39 Solucion de colision de enumerados #40

Merged
merged 5 commits into from
Mar 12, 2024
Merged

PR/39 Solucion de colision de enumerados #40

merged 5 commits into from
Mar 12, 2024

Conversation

Domi-ATLAS
Copy link
Contributor

No description provided.

@Domi-ATLAS Domi-ATLAS self-assigned this Mar 5, 2024
@Domi-ATLAS Domi-ATLAS added backend Issues related to backend development bug Something isn't working code Tasks related to coding labels Mar 5, 2024
ocial/models.py Outdated
Comment on lines 4 to 9
class Category(Enum):
SPORTS = 0 # , ("Sports")
MUSIC = 1 # , ("Music")
MARKETS = 2 # , ("Markets")
RELAX_ACTIVITIES = 3 # , ("Relax activities")
LIVE_CONCERT = 4 # , ("Live concert")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Igual sería mejor ponerlo solo como texto en ambos lados (o igual a la izquierda numero a la derecha texto funciona mejor?)

Me acabo de dar cuenta de esto que me genera el cliente:
image

image

Copy link
Contributor

@ferferga ferferga left a 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).

Comment on lines +5 to +9
SPORTS = "Sports" # , ("Sports")
MUSIC = "Music" # , ("Music")
MARKETS = "Markets" # , ("Markets")
RELAX_ACTIVITIES = "Relax activities" # , ("Relax activities")
LIVE_CONCERT = "Live concert" # , ("Live concert")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
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"

Copy link
Contributor

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

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

dropeado

@adrrf adrrf self-assigned this Mar 11, 2024
@adrrf adrrf force-pushed the fix/39-open-api branch from 845686a to 648f570 Compare March 11, 2024 08:08
Copy link

Quality Gate Passed Quality Gate passed

Issues
2 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

Copy link
Contributor

@ferferga ferferga left a 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.

@ferferga ferferga dismissed their stale review March 11, 2024 23:59

Generación API correcta, falta eliminación de los comentarios para mi gusto, pero no es un blocker para mí

@adrrf adrrf merged commit db581d3 into develop Mar 12, 2024
1 check passed
@adrrf adrrf deleted the fix/39-open-api branch March 12, 2024 07:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backend Issues related to backend development bug Something isn't working code Tasks related to coding
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants