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.

Sign up for GitHub

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

Jump to bottom

fix: update path to the la velada logo in the README #431

Merged
merged 2 commits into from
Mar 11, 2024
Merged

fix: update path to the la velada logo in the README #431

merged 2 commits into from
Mar 11, 2024

Conversation

el-frontend
Copy link
Contributor

Descripción

Actualizado logo del La Velada en el README

Problema solucionado

El logo apuntaba a un path que no existia

Cambios propuestos

Se propone usar el logo que esta en este path /public/img/drawn-x-logo.webp

Capturas de pantalla (si corresponde)

image

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.

@vercel Vercel
Copy link

vercel bot commented Mar 11, 2024 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
la-velada-web-oficial ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 11, 2024 7:01pm

@AlejandroSuero
Copy link
Contributor

Hay una PR abierta con ese problema #426.

@el-frontend
Copy link
Contributor Author

Excelente, valoren usar esta imagen que he puesto en este pr

@AlejandroSuero
Copy link
Contributor

Sí lo pondré en como referencia, de momento, es como estaba antes, pero el link aparece roto.

@AlejandroSuero AlejandroSuero mentioned this pull request Mar 11, 2024
Closed
4 tasks
@github-actions GitHub Actions
Copy link

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

@github-actions GitHub Actions
Copy link

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

@github-actions GitHub Actions
Copy link

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

@github-actions GitHub Actions
Copy link

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

@github-actions GitHub Actions
Copy link

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

@github-actions GitHub Actions
Copy link

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

@github-actions GitHub Actions
Copy link

✅ ¡Los conflictos han sido resuletos! Un colaborador revisará pronto la Pull Request.

@joelmh-112
Copy link
Collaborator

Holaa!! Me parece muy buena idea usar el logo nuevo.

He resuelto los conflictos y procedo a aceptar PR.

A seguir moviendo esas manitaas 🚀

@joelmh-112 joelmh-112 merged commit f6c92e8 into midudev:main Mar 11, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@el-frontend @AlejandroSuero @joelmh-112