search

humitos / osm-pois

Show POIs from OSM in a map using OverpassAPI
http://upoi.org/
GNU General Public License v2.0
22 stars 21 forks source link

Upoi django #74

Closed Elena-GHub closed 3 years ago

Elena-GHub commented 3 years ago

Merge into master: [edit 2021-01-19: changed PR merge into django instead of into master]

Elena-GHub commented 3 years ago

Basic migration OK

Elena-GHub commented 3 years ago

Hola @Elena-GHub! No entiendo bien qué pasó con tus cambios, pero por algún motivo no se ven reflejados en este Pull Request. Al parecer todos los commits que mencionás están en una rama de fork y que no son parte de esta Pull Request.

Debes hacer los cambios en tu branch local upoi-django, que está asociada a esta Pull Request. Luego al hacer push a esa branch, los cambios se van a ver reflejados en esta Pull Request y los voy a poder revisar. ¿Puedes mirar esto por favor?

Tip: siempre es bueno ver la pestaña "Files changed" del Pull Request sobre el que estás trabajando para estar segura de que los cambios propuestos son los que esperas enviar.

Perdona @humitos. Seguramente no le di al botón adecuado o en el orden adecuado, estos son mis primeros tanteos de intercambios a través de una PR. Cuando vuelva a casa y me reúna de nuevo con @EllaQuímica, lo repasamos.

Elena-GHub commented 3 years ago

Hola @Elena-GHub! No entiendo bien qué pasó con tus cambios, pero por algún motivo no se ven reflejados en este Pull Request. Al parecer todos los commits que mencionás están en una rama de fork y que no son parte de esta Pull Request.

Debes hacer los cambios en tu branch local upoi-django, que está asociada a esta Pull Request. Luego al hacer push a esa branch, los cambios se van a ver reflejados en esta Pull Request y los voy a poder revisar. ¿Puedes mirar esto por favor?

Tip: siempre es bueno ver la pestaña "Files changed" del Pull Request sobre el que estás trabajando para estar segura de que los cambios propuestos son los que esperas enviar.

¡Hola, @humitos Manuel! (@EllaQuimica) Gracias por el consejo de fijarnos en la rama. Hicimos el merge de master a upoi-django y ya figuran aquí los commits que resuelven cada punto de esta conversación. Esperamos tu visto bueno o comentarios adicionales. Ciao!

Elena-GHub commented 3 years ago 
* some assets files that are not loading correctly. I'm getting 404 on those

Fixed by correcting path: