Plots2: Adicionar recurso de tag de redirecionamento por nome de página

Olá, este é um problema apenas para os 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.

Temos um sistema de redirecionamento que usa tags - leia mais em https://publiclab.org/wiki/power-tags - mas requer que as tags apontem para um "id do nó" como redirect:12345 . Isso é difícil de lembrar e meio obscuro - então, vamos possibilitar o uso de um "slug" de página também. Então você pode redirecionar para a página /wiki/openhour usando redirect:openhour .

📋 Passo a passo

• [] 🙋 Reivindique este problema : Comentário abaixo. Se outra pessoa a reivindicou, pergunte se ela já abriu uma solicitação de pull e se ela está travada - talvez você possa ajudá-la a resolver um problema ou seguir em frente!

• [] 📝 Atualize o arquivo app / controllers / wiki_controller.rb no plots2 (pressione o pequeno ícone de caneta) e edite a linha como mostrado abaixo.

Consulte esta página para obter ajuda para dar os primeiros passos!

Abaixo está uma "diferença" mostrando em vermelho (e - ) quais linhas remover e em verde (e + ) quais linhas adicionar:

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

• [] 💾 Comprometa suas mudanças

• [] 🔀 Inicie uma solicitação pull . Existem duas maneiras de iniciar uma solicitação pull:

1. Se você está familiarizado com o terminal ou gostaria de aprendê-lo, aqui está um ótimo tutorial sobre como enviar uma solicitação de pull usando o terminal.

2. Você também pode editar arquivos diretamente em seu navegador e abrir uma solicitação de pull de lá.

• [] 🏁 Concluído Peça nos comentários para uma revisão :)

🤔❓ Dúvidas?

Deixe um comentário abaixo!

Alguém já está trabalhando nisso?

Incentivamos você a fazer um link para este problema mencionando o problema # em sua solicitação de pull, para que possamos ver se alguém já começou a fazer isso. Se alguém parecer travado, ofereça ajuda! Caso contrário, dê uma olhada em alguns outros problemas em que você pode ajudar . Obrigado!

(Este problema foi criado pelo First-Timers-Bot .)