zoom-is-configurable

[delineas]

Cómo programador
Quiero modificar el zoom por defecto
Para que sea un parámetro configurable

Criterios:

• Añadir el parámetro zoom en https://github.com/delineas/help-maps-party/blob/master/src/settings.js
• Traerlo como valor de la configuración en la inicialización del mapa en https://github.com/delineas/help-maps-party/blob/master/src/components/Map.vue#L62

[skcode7]

Saludos Dani
Me apunto para colaborar, voy a revisar un poco el código y luego crearme una rama a ver como me vá.

[skcode7]

Que tal Dani,
Te comento que ya hice el cambio y funciona perfecto en local, ahora tengo una duda con respecto al pull-request
Ayer estuve haciendo algunos cambios y los respectivos commits, hoy luego del cambio del zoom hice un nuevo commit y al generar el PR aparecen todos los objetos modificados.
Estuve investigando un poco y al parecer si se puede eliminar los commits, pero no me queda claro el proceso.

[delineas]

La mejor opción es esta:
https://mattstauffer.com/blog/how-to-merge-only-specific-commits-from-a-pull-request/

Con esto lo que harías sería tener en una rama el cambio específico que quieres enviar al repositorio padre.

Eso sí, necesitas poner el repositorio en el mismo punto en el que estaba en origen, antes de introducir tus modificaciones. Esto es, deberías tener una rama (master, por ejemplo) donde no añadirías modificaciones tuyas salvo que quieras que formen parte el repositorio padre.

Todo esto es para evitar pasar de un repo a otro en el PR los cambios que son específicos de tu proyecto.

De esta aprendemos todos :))

[skcode7]

Finalmente lo logre! :)
Me tomó varias horas entender los pasos a seguir, pero al final quedó el PR con los cambios correctos.
Bien dicho, aprendemos todos.

[delineas]

Gracias! Y felicidades! Ahora me toca a mi repasar lo que has enviado y practicar una saludable y positiva Code Review ;)