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

Sponsorsfromdb #68

Merged
merged 6 commits into from
Sep 20, 2024
Merged

Conversation

cgarros33
Copy link
Contributor

  • Updated sponsors component to fetch from Firebase and remove deprecated exports.
  • Removed sponsors data from repo as it is now stored in Firebase.

msambartolomeo and others added 6 commits February 19, 2024 21:09
Changes:
 - Update carrousel with new dependency, should be enough to fix the UI
   bugs presented in the page.
 - Update sponsors section.
…A/develop

[UI/UX] Update Upcoming Event and Team Members
Copy link
Contributor

@FSendot FSendot left a comment

Choose a reason for hiding this comment

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

Los cambios están bien, responde como es debido. Falta una última cosa nada más, hay un tema con los textos alt, que están internacionalizados, si te fijás bien en el directorio i18n del proyecto, se encuentran esos textos alternativos. De dejarlo así, cuando alguien use el importer para actualizar los imágenes, van a tener un problema y van a necesitar actualizar a mano los textos alternativos, internacionalizados, para cada sponsor agregado o eliminado.
Se me ocurren dos opciones para esto:
Podemos hacer que el texto alternativo sea genérico, esto es, al indicar el texto alternativo usando i18n, que se acceda al campo mediante una llamada de este estilo, por ejemplo:

<img src-... ={i18n.t('sponsors.logos')} ...>

Ahora, una manera de que el texto del logo sea genérico, se puede formatear de manera tal que el logo tenga una estructura de este estilo:

sponsors: {
   title: 'Nuestros sponsors'
   logos: 'Logo de {}'
}

Finalmente, en el texto alternativo, el string quedaría formateado de la siguiente forma:

<img src-... ={i18n.t('sponsors.logos').format(sponsor.name)} ...>

Para todo esto sería necesario quizás modificar el esquema que se hizo en Firebase, agregando dicho campo (y eliminando el de alt), capaz para alivianar el tema del Firebase, podrías renombrar el campo alt por name (total ya no tiene sentido que ese campo esté en Firebase).
Esto implicaría actualizar el importer a futuro para que se adapte a este cambio nuevo.

@FSendot FSendot merged commit d6e94e7 into Computer-Society-ITBA:develop Sep 20, 2024
1 check failed
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.

3 participants