Plots2: Ajouter une fonctionnalité de balise de redirection par nom de page

Bonjour, 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 est destiné à vous souhaiter la bienvenue dans l'Open Source :) Nous sommes heureux de vous guider tout au long du processus.

Nous avons un système de redirection qui utilise des balises - en savoir plus sur https://publiclab.org/wiki/power-tags - mais il nécessite que les balises pointent vers un "identifiant de nœud" comme redirect:12345 . C'est difficile à retenir et un peu obscur -- rendons donc possible l'utilisation d'un "slug" de page également. Ensuite, vous pouvez rediriger vers la page à /wiki/openhour en utilisant redirect:openhour .

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é. Peut-être pouvez-vous l'aider à résoudre un problème ou à le déplacer !

• [ ] 📝 Mettez à jour le fichier app/controllers/wiki_controller.rb dans le référentiel plots2 (appuyez sur l'icône du petit stylo) et modifiez la ligne comme indiqué ci-dessous.

Consultez cette page pour obtenir de l'aide pour faire vos premiers pas!

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

@@ -33,6 +33,14 @@ def show
         flash.now[:warning] = "Only moderators and admins see this page, as it is redirected to <a href='#{Node.find(@node.power_tag('redirect')).path}'>#{Node.find(@node.power_tag('redirect')).title}</a>.
         To remove the redirect, delete the tag beginning with 'redirect:'"
       end
+    elsif @node&.has_power_tag('redirect') && Node.where(slug: @node.power_tag('redirect')).exists?
+      if current_user.nil? || !current_user.can_moderate?
+        redirect_to Node.find_by(slug: @node.power_tag('redirect')).path
+        return
+      elsif current_user.can_moderate?
+        flash.now[:warning] = "Only moderators and admins see this page, as it is redirected to <a href='#{Node.find_by(slug: @node.power_tag('redirect')).path}'>#{Node.find_by(slug: @node.power_tag('redirect')).title}</a>.
+        To remove the redirect, delete the tag beginning with 'redirect:'"
+      end
     end

     if @node&.has_power_tag('abtest') && !Node.where(nid: @node.power_tag('abtest')).empty?

• [] 💾 vos changements

• [ ] 🔀 Lancer une Pull Request . Il existe deux manières de lancer une pull request :

1. Si vous connaissez le terminal ou si vous souhaitez l'apprendre, voici un excellent tutoriel sur la façon d'envoyer une pull request à l'aide du terminal.

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

• [ ] 🏁 Terminé Demandez un avis dans les commentaires :)

Des questions ?

Laissez un commentaire ci-dessous !

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

Nous vous encourageons à créer un lien vers ce problème en mentionnant le numéro du problème dans votre pull request, afin que nous puissions voir si quelqu'un a déjà commencé à le traiter. Si quelqu'un semble bloqué, proposez-lui de l'aide ! Sinon, jetez un œil à d'autres problèmes pour lesquels vous pouvez aider . Merci!

(Ce problème a été créé par First-Timers-Bot .)