Plots2: Refactoring profile.html.erb code pt.2

Première fois?

Il s'agit d'un problème réservé aux débutants . Cela signifie que nous avons travaillé pour le rendre plus lisible pour les personnes qui n'ont jamais contribué à notre base de code auparavant, ou même pour celles qui n'ont jamais contribué à l'open source auparavant .

Si tel est votre cas, nous souhaitons vous aider à faire le premier pas et pouvons répondre à vos questions et vous aider comme vous le faites. Notez que nous sommes particulièrement intéressés par les contributions de personnes issues de groupes sous-représentés dans les logiciels libres et open source !

Nous savons que le processus de création d'une pull request est le plus gros obstacle pour les nouveaux contributeurs. Ce numéro est pour vous

Si vous avez déjà contribué, pensez à laisser celui-ci à quelqu'un de nouveau et à consulter notre aide générale recherchés . Merci!

Ce que vous devez savoir.

Rien. Ce numéro a pour but de vous souhaiter la bienvenue dans l'Open Source :) Nous sommes heureux de vous guider tout au long du processus.

Le problème

Notre code dans app/views/users/profile.html.erb est très verbeux... nous pourrions utiliser une refactorisation

Nous pouvons commencer par refactoriser une ligne récurrente du code. Pour l'instant, la 1ère ligne de :

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

Solution

En quoi le changer :

Refactorisez la ligne dans le bloc de code trouvé dans la section "Le problème" pour utiliser votre nouvelle méthode d'assistance à la place.

Cette ligne peut être mise à jour en <% if current_profile_user %>

Merci!!

Étapes à corriger

• [ ] Réclamez ce problème avec un commentaire ici, ci-dessous, et posez toutes les questions de clarification dont vous avez besoin
• [ ] Forkez le référentiel et configurez-le localement en suivant les instructions README du référentiel principal https://github.com/publiclab/plots2
  
  
  
  • [ ] Créer une nouvelle branche de fonctionnalité avec un nom unique décrivant le problème
  
  
• [ ] Essayez de résoudre le problème en suivant les étapes ci-dessus, mais avant même d'avoir terminé, vous pouvez :
  validez vos modifications dans votre branche et lancez une pull request (voir contribution au logiciel Public Lab ) mais marquez-la comme "en cours" si vous avez des questions ou si vous n'avez pas terminé
• [ ] Référencez ce problème dans le corps de votre pull request
• [ ] Une fois que vous avez soumis votre pull request, une liste de contrôle supplémentaire sera fournie pour la fusionner

Obtenez de l'aide

Si vous avez besoin d'aide, voici quelques options :

• Commentaires ci-dessous
• Rejoignez notre chat gitter sur https://gitter.im/publiclab/publiclab