Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix:Descripcion #25

Merged
merged 3 commits into from
Jul 26, 2023
Merged

Fix:Descripcion #25

merged 3 commits into from
Jul 26, 2023

Conversation

Ercamarero
Copy link
Collaborator

@Ercamarero Ercamarero commented Jul 26, 2023

Description

No se estaba añadiendo la descripcion ni se estaba actualizando en ningun metodo.

Commit type

Issue

Al no crear bien la playlist se rompia el server

Solution

Proposed Changes

  • Cambio en los parametros del main en Update.
  • Cambio en update en playlist_services,
  • Proteccion del dato imagen al actualizar la playlist
  • Devolucion del dato descripccion
  • Proteccion dato imagen
  • Insercion de datos en la clase

Potential Impact

  • Los métodos de update playlist requerirán parámetro description

Tests Performed

  • [✅] Test performed

Screenshots

Additional Tasks

Assigned

@AntonioMrtz

Copy link
Owner

@AntonioMrtz AntonioMrtz left a comment

Choose a reason for hiding this comment

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

En impacto potencial , te ha faltado que las llamadas a métodos update de playlist ahora deberán llevar un nuevo parametro description

@AntonioMrtz AntonioMrtz merged commit f104d9a into master Jul 26, 2023
@AntonioMrtz AntonioMrtz deleted the Descripccion-fix branch July 26, 2023 22:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants