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

Fix #27 Se modificó index.md para que la aplicación levante en desarrollo #28

Merged
merged 1 commit into from Jun 11, 2020

Conversation

pablogassman
Copy link
Contributor

Se elimino la linea que tiraba el error.

@PalumboN
Copy link
Collaborator

PalumboN commented Jun 7, 2020

Hola @pablogassman

Uy que bueno que detectaste esto. Esa línea la necesitamos para la página que está en producción (https://uqbar-project.github.io/function-laboratory/), en realidad es algo hack porque todavía no nos pusimos a pulir la página que nos da GitHub.

Quizá se podría seguir buscando un workaround y meter esa línea en un if (document.getElementsByClassName("wrapper")[0]) para no sacarla completamente?

@PalumboN
Copy link
Collaborator

PalumboN commented Jun 7, 2020

Ahh y cuando tires un PR está bueno hacer referencia al issue en el comentario, de esta forma:

Fix #27

Así ya queda linkeado y se cierra solo cuando se mergea el PR ;)

@PalumboN PalumboN linked an issue Jun 7, 2020 that may be closed by this pull request
@pablogassman pablogassman changed the title Se elimino la linea que generaba conflicto e impedía que la aplicación levantara Fix #27 Se modificó index.md para que la aplicación levante en desarrollo Jun 9, 2020
@pablogassman
Copy link
Contributor Author

Hola @PalumboN
Ya modifique el index.md siguiendo tu sugerencia.

@PalumboN PalumboN merged commit 32936cb into uqbar-project:master Jun 11, 2020
@PalumboN
Copy link
Collaborator

Buenísimo! Muchas gracias @pablogassman

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.

Luego de correr "npm start" la aplicación no levanta.
2 participants