Plots2: Ajouter des balises dans les modaux des nœuds dans Spam2

Créé le 4 août 2020  ·  3Commentaires  ·  Source: publiclab/plots2

Bonjour, c'est 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 les personnes qui n'ont jamais contribué à l'open source auparavant .

Si c'est vous, nous sommes intéressés à vous aider à faire le premier pas et pouvons répondre aux questions et vous aider comme vous le faites. Notez que nous sommes particulièrement intéressés par les contributions de personnes appartenant à des 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 grand obstacle pour les nouveaux contributeurs. Ce numéro est pour vous 💝

Si vous avez déjà contribué, envisagez de laisser celui-ci à quelqu'un de nouveau et de consulter nos problèmes généraux d' aide recherchés . Merci!

🤔 Ce que vous devez savoir.

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

📋 Pas à pas

  • [ ] 🙋 Réclamez ce numéro : Commentez ci-dessous. Si quelqu'un d'autre l'a revendiqué, demandez-lui s'il a déjà ouvert une pull request et s'il est bloqué. Vous pouvez peut-être l'aider à résoudre un problème ou à le faire avancer !

  • [ ] 📝 Mettez à jour le fichier https://github.com/publiclab/plots2/blob/main/app/views/spam2/_nodes.html.erb#L144 et ajoutez les balises de ce nœud comme indiqué ci-dessous.

                  <div class="modal-body">
                    <% node.tags.limit(4).each do |tag| %>
                      <span ><a class="badge badge-primary" href="/questions/tag/<%= tag.name %>"> <%= tag.name %> </a></span>
                    <% end %>
                    <br>
                    <%= node.body %>
                  </div>

Cela devrait ressembler à ceci :
Screenshot from 2020-08-04 04-36-53

Pour tester ça

  • Connectez-vous en tant que modérateur ou administrateur
  • Remplacez l'URL par /Spam2
  • Dans le menu déroulant, accédez au wiki ou à la modération groupée
  • Cliquez sur n'importe quel titre
  • Vérifier les balises en modal

Consultez cette page pour vous aider à faire vos premiers pas !

Ci-dessous un "diff" montrant en rouge (et un - ) les lignes à supprimer, et en vert (et un + ) les lignes à ajouter :

$DIFF
  • [ ] 💾 Validez vos modifications

  • [ ] 🔀 Démarrez une demande d' extraction. Il existe deux manières de lancer une pull request :

  1. Si vous êtes familier avec le terminal ou si vous souhaitez l'apprendre, voici un excellent tutoriel sur la façon d'envoyer une demande d'extraction à l'aide du terminal.

  2. Vous pouvez également modifier des fichiers directement dans votre navigateur et ouvrir une demande d'extraction à partir de là.

  • [ ] 🏁 Fait Demander en commentaires pour un avis :)

S'il-vous-plait, gardez nous au courant

💬⏰ - Nous encourageons les contributeurs à être respectueux envers la communauté et à fournir une mise à jour dans la semaine suivant la réclamation d'un problème réservé aux débutants. Nous sommes heureux de vous l'attribuer aussi longtemps que vous en avez besoin si vous nous mettez à jour avec une demande de temps ou d'aide supplémentaire, mais si nous ne voyons aucune activité une semaine après que vous l'avez réclamée, nous pouvons la réattribuer pour donner à quelqu'un sinon une chance. Merci d'avance!

Si cela vous arrive, ne vous inquiétez pas ! Prenez un autre problème ouvert.

Est-ce que quelqu'un d'autre travaille déjà là-dessus?

🔗- Nous encourageons les contributeurs à créer un lien vers le problème d'origine dans leur pull request afin que tous les utilisateurs puissent facilement voir si quelqu'un l'a déjà commencé.

👥- Si quelqu'un semble bloqué, proposez-lui de l'aide ! Sinon, jetez un œil à d'autres problèmes que vous pouvez résoudre . Merci!

🤔❓ Des questions ?

Laissez un commentaire ci-dessous!

first-timers-only

Commentaire le plus utile

J'ai soumis une pull request pour ce #8238. @cesswairimu @keshavsethi Veuillez revoir.

Tous les 3 commentaires

Bonjour, je relève ce problème. Merci.

J'ai soumis une pull request pour ce #8238. @cesswairimu @keshavsethi Veuillez revoir.

@keshavsethi J'aimerais aussi savoir où écrire des tests pour le changement que j'ai apporté. Merci.

Cette page vous a été utile?
0 / 5 - 0 notes

Questions connexes

ebarry picture ebarry  ·  3Commentaires

milaaraujo picture milaaraujo  ·  3Commentaires

first-timers[bot] picture first-timers[bot]  ·  3Commentaires

grvsachdeva picture grvsachdeva  ·  3Commentaires

bronwen9 picture bronwen9  ·  3Commentaires