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

Refactor/number of plays #51

Merged
merged 3 commits into from
Aug 22, 2023
Merged

Refactor/number of plays #51

merged 3 commits into from
Aug 22, 2023

Conversation

AntonioMrtz
Copy link
Owner

Description

Refactorizaciónn del endpoint para actualizar los datos de una canciñon de manera que no es necesario actualizar el objeto entero y la llamada requiere menos tiempo.

Commit type

Refactor

Issue

  • La operación de actualizar el objeto canción se llevaba acabo solo para aumentar un campo.
  • La función Update en canciones/ tenia demasiadas responsabilidades.

Solution

Creación método patch que solo actualiza el campo de las reproducciones en la cancion seleccionada.

Proposed Changes

  • Extración en método PATCH para actualizar el número de canciones
  • Test comprobación método UPDATE y PATCH
  • Actualización código frontend con los cambios efectuados

Potential Impact

Tests Performed

  • [✅] Test performed

Screenshots

Additional Tasks

Assigned

@AntonioMrtz

@AntonioMrtz AntonioMrtz merged commit 8fa3e2b into master Aug 22, 2023
@AntonioMrtz AntonioMrtz deleted the refactor/NumberOfPlays branch August 22, 2023 20:17
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.

1 participant