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

UCalendar en RamosUC #104

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

UCalendar en RamosUC #104

wants to merge 3 commits into from

Conversation

benjavicente
Copy link
Member

@benjavicente benjavicente commented Jan 4, 2023

Chan

Murió el deploy por Heroku, y logré implementar la versión de UCalendar con cosas hard-codeadas con 1 archivo (al final lo modularicé) de JS que corra en el frontend.

Falta probarlo más antes de unirlo (me funciona bien 🚀 ), y además limpiar un poco el código (listo) y darle más color al botón(listo, con una gradiente para que se destaque).

También podríamos ver como se eliminan esas cosas hard-codeadas, pero para alcanzar a lanzarlo en este semestre, no lo haría completamente necesario para esta PR.

El botón

@benjavicente benjavicente added feature New feature or request javascript Pull requests that update Javascript code labels Jan 4, 2023
@benjavicente benjavicente self-assigned this Jan 4, 2023
@benjavicente benjavicente marked this pull request as ready for review January 7, 2023 20:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request javascript Pull requests that update Javascript code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant