Plots2: Refactorización del código profile.html.erb pt.2

Creado en 19 dic. 2018  ·  4Comentarios  ·  Fuente: publiclab/plots2

¿Primera vez?

Este es un problema exclusivo para principiantes . Esto significa que hemos trabajado para hacerlo más legible para las personas que no han contribuido antes con nuestro código base, o incluso para las personas que no han contribuido antes con el código abierto .

Si ese es usted, estamos interesados ​​en ayudarlo a dar el primer paso y podemos responder preguntas y ayudarlo mientras lo hace. Tenga en cuenta que estamos especialmente interesados ​​en las contribuciones de personas de grupos subrepresentados en software gratuito y de código abierto.

Sabemos que el proceso de creación de una solicitud de extracción es la mayor barrera para los nuevos contribuyentes. Este problema es para ti 💝

Si ha contribuido antes, considere dejar este para alguien nuevo y revise nuestros problemas de búsqueda de ayuda general. ¡Gracias!

🤔 Lo que necesita saber.

Nada. Este número está destinado a darle la bienvenida a Open Source :) Nos complace guiarlo a través del proceso.

El problema

Nuestro código en app/views/users/profile.html.erb es muy detallado ... podríamos usar un poco de refactorización

Podemos comenzar refactorizando una línea que vuelva a ocurrir del código. Por ahora, la 1ra línea de:

https://github.com/publiclab/plots2/blob/1931439c0c799b3e3420a9dcaace7a54cf895bd4/app/views/users/profile.html.erb#L216 -L223

Solución

A qué cambiarlo:

Refactorice la línea en el bloque de código que se encuentra en la sección "El problema" para usar su nuevo método auxiliar.

Esa línea se puede actualizar a <% if current_profile_user %>

¡¡Gracias!!

Pasos para arreglar

  • [] Reclame este problema con un comentario aquí, a continuación, y haga las preguntas aclaratorias que necesite.
  • [] Bifurque el repositorio y configúrelo localmente siguiendo las instrucciones README del repositorio principal https://github.com/publiclab/plots2

    • [] Cree una nueva rama de funciones con un nombre único que describa el problema.

  • [] Intente solucionar el problema siguiendo los pasos anteriores, pero incluso antes de que termine, puede:
    Confirme sus cambios en su sucursal e inicie una solicitud de extracción (consulte Contribución al software Public Lab ), pero márquelo como "en progreso" si tiene preguntas o si no ha terminado.
  • [] Haga referencia a este problema en el cuerpo de su solicitud de extracción.
  • [] Una vez que envíe su solicitud de extracción, se le proporcionará una lista de verificación adicional para fusionarla.

💬 Obtenga ayuda

Si necesita ayuda, aquí hay algunas opciones:

first-timers-only help wanted

Comentario más útil

Todavía tengo que hacer una contribución de código abierto para que esto me caliente. ¿Puedo reclamar esto?

Todos 4 comentarios

Todavía tengo que hacer una contribución de código abierto para que esto me caliente. ¿Puedo reclamar esto?

Hola @ dave-light, adelante. Y bienvenido a Public Lab y Open Source World: globe_with_meridians:: earth_americas:.

@ dave-light: ¿estás en eso? o puede reclamar esto?

Hola @mzmmlds , estoy en eso.

¿Fue útil esta página
0 / 5 - 0 calificaciones