Plots2: Agregue la función de etiqueta de redireccionamiento por nombre de página

Hola, este es un problema solo 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 barrera más grande 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.

Tenemos un sistema de redireccionamiento que usa etiquetas - lea más en https://publiclab.org/wiki/power-tags - pero requiere que las etiquetas apunten a un "ID de nodo" como redirect:12345 . Esto es difícil de recordar y algo oscuro, así que hagamos posible el uso de un "slug" de página también. Luego, podría redirigir a la página en /wiki/openhour usando redirect:openhour .

📋 Paso a paso

• [] 🙋 Reclame este problema : Comente a continuación. Si alguien más lo ha reclamado, pregúntele si ya ha abierto una solicitud de extracción y si está atascado, ¡tal vez pueda ayudarlo a resolver un problema o moverlo!

• [] 📝 Actualice el archivo app / controllers / wiki_controller.rb en el repositorio plots2 (presione el ícono del bolígrafo) y edite la línea como se muestra a continuación.

¡Consulte esta página para obtener ayuda para dar sus primeros pasos!

A continuación se muestra una "diferencia" en rojo (y un - ) qué líneas eliminar, y en verde (y un + ) qué líneas agregar:

@@ -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?

• [] 💾 confirmar los cambios

• [] 🔀 Inicie una solicitud de extracción. Hay dos formas de iniciar una solicitud de extracción:

1. Si está familiarizado con el terminal o le gustaría aprenderlo, aquí hay un gran tutorial sobre cómo enviar una solicitud de extracción utilizando el terminal.

2. También puede editar archivos directamente en su navegador y abrir una solicitud de extracción desde allí.

• [] 🏁 Listo Pregunta en los comentarios para una revisión :)

🤔❓ ¿Preguntas?

¡Deja un comentario a continuación!

¿Alguien más ya está trabajando en esto?

Le recomendamos que se vincule a este problema mencionando el número de problema en su solicitud de extracción, para que podamos ver si alguien ya ha comenzado. Si alguien parece estancado, ¡ofrézcale ayuda! De lo contrario, eche un vistazo a algunos otros problemas con los que puede ayudar . ¡Gracias!

(Este problema fue creado por First-Timers-Bot .)