Plots2: Добавить функцию тега перенаправления по имени страницы

Привет, это проблема только для новичков . Это означает, что мы работали над тем, чтобы сделать его более понятным для людей, которые либо не вносили вклад в нашу кодовую базу раньше, либо даже для людей, которые раньше не участвовали в разработке открытого исходного кода .

Если это вы, мы заинтересованы в том, чтобы помочь вам сделать первый шаг, ответить на вопросы и помочь вам, как и вы. Обратите внимание, что мы особенно заинтересованы в участии людей из групп, недостаточно представленных в бесплатном программном обеспечении с открытым исходным кодом!

Мы знаем, что процесс создания запроса на вытягивание является самым большим препятствием для новых участников. Этот выпуск для вас 💝

Если вы вносили свой вклад раньше, подумайте о том, чтобы оставить это кому-то новому и просмотреть наши общие справочные вопросы. Спасибо!

🤔 Что вам нужно знать.

Ничего такого. Этот выпуск предназначен для того, чтобы поприветствовать вас в Open Source :) Мы будем рады помочь вам в этом процессе.

У нас есть система перенаправления, которая использует теги - подробнее на https://publiclab.org/wiki/power-tags - но требует, чтобы теги указывали на «идентификатор узла», например redirect:12345 . Это трудно запомнить и что-то непонятно, так что давайте также сделаем возможным использование "slug" страницы. Затем вы можете перенаправить на страницу /wiki/openhour с помощью redirect:openhour .

📋 Шаг за шагом

• [] 🙋 Заявите о проблеме : комментарий ниже. Если кто-то еще заявил об этом, спросите, открывал ли он уже пул-реквест и застрял ли он - может быть, вы сможете помочь им решить проблему или продвинуть ее дальше!

• [] 📝 Обновите файл app / controllers / wiki_controller.rb в plots2 (нажмите маленький значок ручки) и отредактируйте строку, как показано ниже.

Посетите эту страницу, чтобы

Ниже приведена «разница», показывающая красным (и - ) строки, которые нужно удалить, и зеленым (и + ) строки, которые нужно добавить:

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

• [] 💾 Зафиксируйте изменения

• [] 🔀 Запустить запрос на включение. Есть два способа запустить пул-реквест:

1. Если вы знакомы с терминалом или хотите его изучить, вот отличное руководство о том, как отправить запрос на перенос с помощью терминала.

2. Вы также можете редактировать файлы прямо в браузере и оттуда открывать запрос на перенос.

• [] 🏁 Готово Попросите в комментариях оставить отзыв :)

🤔❓ Вопросы?

Оставьте комментарий ниже!

Кто-то уже работает над этим?

Мы рекомендуем вам указать ссылку на эту проблему, указав номер проблемы в своем запросе на перенос, чтобы мы могли увидеть, начал ли кто-то уже над этим. Если кто-то застрял, предложите ему помощь! В противном случае обратите внимание на другие проблемы, с которыми вы можете помочь . Спасибо!

(Эта проблема была создана новичком-ботом .)