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

feat: Add "Ver combate" button to Combates page #971

Closed
wants to merge 3 commits into from
Closed

feat: Add "Ver combate" button to Combates page #971

wants to merge 3 commits into from

Conversation

GFrancV
Copy link
Contributor

@GFrancV GFrancV commented Jul 15, 2024

Descripción

Se agregó el botón "Ver Combate" en la página de los combates con el link directo al video de Youtube de cada combate. Además, se eliminó el botón "Haz tu pronóstico" puesto que una vez terminada la velada no hace mucho sentido que siga habiendo la posibilidad de hacer predicciones.

Cambios propuestos

  1. Añadir boton "Ver combate" con el link para el video correspondiende de YouTube de cada combate.
  2. Retirar el boton de "Haz tu pronostico" en la pagina de cada combate.

Comprobación de cambios

  • He revisado que no haya ninguna PR (pull request) ya abierta con un problema similar, siguiendo el apartado de buenas prácticas
  • He revisado localmente los cambios para asegurarme de que no haya errores ni problemas.
  • He probado estos cambios en múltiples dispositivos y navegadores para asegurarme de que la landing page se vea y funcione correctamente.
  • He actualizado la documentación, si corresponde.

Copy link

vercel bot commented Jul 15, 2024

@GFrancV is attempting to deploy a commit to the midudev pro Team on Vercel.

A member of the Team first needs to authorize it.

@GFrancV GFrancV changed the title feat: Add "Ver combate" button to Combates page #970 feat: Add "Ver combate" button to Combates page Jul 15, 2024
Copy link

@pyoneerC pyoneerC left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link

⚠️ Esta Pull Request tiene conflictos. Por favor, resuélvelos antes de que podamos evaluar los cambios.

@GFrancV GFrancV closed this Jul 17, 2024
@GFrancV GFrancV deleted the add-see-combat-btn branch July 17, 2024 08:41
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.

2 participants