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

removing external calls to libraries #64

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

removing external calls to libraries #64

wants to merge 1 commit into from

Conversation

laureanoSilva
Copy link

editing the index to make the libraries be local. based in: https://github.com/51114u9/osm-pois/blob/master/web/index.html

@@ -289,17 +289,6 @@
en alguno de los POIs que aparecen en el mapa luego de
seleccionar algunos de la lista.</p>

<p>También puedes crear tu propia consulta utilizando
Copy link
Owner

Choose a reason for hiding this comment

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

¿Esta sección la quitaste por algo en particular o se te olvidó volver a agregarla en otra parte?

@humitos
Copy link
Owner

humitos commented Nov 25, 2015

¿Probaste los cambios que hiciste localmente en tu pc? ¿Te funcionó? Si no, ¿qué problemas tuviste?

Es muy importante que antes de hacer un PR puedas probar los cambios en tu equipo así no enviás algo que no funciona.

Por ejemplo, yo apliqué los 2 PR que enviaste y veo que Firefox me da algunos errores:

screenshot - 251115 - 12 29 15

Para abrir esa ventana con información de depuración, utilicé Control+Shift+K y luego refresqué la página. Ahí se pueden ver errores diciendo que no encuentra Leaflet (la librería para mostrar el mapa) y que no puede descargar algunas fuentes de FontAwesome.

¿Podrías resolver eso y volver a enviar el PR?

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