Plots2: Refatorando profile.html.erb código pt.2

Criado em 19 dez. 2018  ·  4Comentários  ·  Fonte: publiclab/plots2

Primeira vez?

Este é um problema apenas para iniciantes . Isso significa que trabalhamos para torná-lo mais legível para as pessoas que não contribuíram com nossa base de código antes, ou mesmo para as pessoas que não contribuíram com o código aberto antes .

Se for você, estamos interessados ​​em ajudá-lo a dar o primeiro passo e podemos responder a perguntas e ajudá-lo como você faz. Observe que estamos especialmente interessados ​​em contribuições de pessoas de grupos sub-representados em software livre e de código aberto!

Sabemos que o processo de criação de uma solicitação pull é a maior barreira para novos contribuidores. Este problema é para você 💝

Se você já contribuiu antes, considere deixar este para alguém novo e olhar através de nossos problemas de necessidade de ajuda geral. Obrigado!

🤔 O que você precisa saber.

Nada. Esta edição tem como objetivo dar-lhe as boas-vindas ao Open Source :) Estamos felizes em orientá-lo durante o processo.

O problema

Nosso código em app/views/users/profile.html.erb é muito detalhado ... poderíamos usar alguma refatoração

Podemos começar refatorando uma linha recorrente do código. Por enquanto, a 1ª linha de:

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

Solução

O que mudar para:

Refatore a linha no bloco de código encontrado na seção "O problema" para usar seu novo método auxiliar.

Essa linha pode ser atualizada para <% if current_profile_user %>

Obrigado!!

Passos para corrigir

  • [] Reivindique este problema com um comentário aqui, abaixo, e faça as perguntas de esclarecimento de que precisar
  • [] Bifurque o repositório e configure-o localmente seguindo as instruções README do repo principal https://github.com/publiclab/plots2

    • [] Crie um novo branch de recurso com um nome único descritivo para o problema

  • [] Tente corrigir o problema seguindo as etapas acima, mas mesmo antes de terminar, você pode:
    confirme suas alterações em seu branch e inicie uma solicitação de pull (consulte contribuição para o software Public Lab ), mas marque-o como "em andamento" se tiver dúvidas ou se não tiver terminado
  • [] Faça referência a este problema em seu corpo de solicitação pull
  • [] Depois de enviar sua solicitação pull, uma lista de verificação adicional será fornecida para mesclá-la

💬 Obtenha ajuda

Se precisar de ajuda - aqui estão algumas opções:

first-timers-only help wanted

Comentários muito úteis

Ainda estou para fazer uma contribuição de código aberto, então isso me aqueceria. Posso reivindicar isso?

Todos 4 comentários

Ainda estou para fazer uma contribuição de código aberto, então isso me aqueceria. Posso reivindicar isso?

Olá @ dave-light, por favor, vá em frente. E, bem-vindo ao Public Lab e Open Source World: globe_with_meridians:: earth_americas:.

@dave-light: você está trabalhando nisso? ou pode reivindicar isso?

Ei @mzmmlds , estou indo.

Esta página foi útil?
0 / 5 - 0 avaliações

Questões relacionadas

RuthNjeri picture RuthNjeri  ·  3Comentários

divyabaid16 picture divyabaid16  ·  3Comentários

keshavsethi picture keshavsethi  ·  3Comentários

milaaraujo picture milaaraujo  ·  3Comentários

jywarren picture jywarren  ·  3Comentários