Plots2: Tambahkan fitur tag redirect dengan nama halaman

Hai, ini hanya masalah pemula . Ini berarti kami telah bekerja untuk membuatnya lebih mudah dibaca oleh orang-orang yang belum pernah berkontribusi pada basis kode kami sebelumnya, atau bahkan orang-orang yang belum pernah berkontribusi pada open source sebelumnya .

Jika itu Anda, kami tertarik untuk membantu Anda mengambil langkah pertama dan dapat menjawab pertanyaan serta membantu Anda seperti yang Anda lakukan. Perhatikan bahwa kami sangat tertarik pada kontribusi dari orang-orang dari kelompok yang kurang terwakili dalam perangkat lunak bebas dan sumber terbuka!

Kami tahu bahwa proses pembuatan pull request adalah hambatan terbesar bagi kontributor baru. Masalah ini untukmu

Jika Anda telah berkontribusi sebelumnya, pertimbangkan untuk meninggalkan yang satu ini untuk seseorang yang baru , dan mencari masalah yang diinginkan bantuan umum kami. Terima kasih!

Apa yang perlu Anda ketahui.

Tidak. Masalah ini dimaksudkan untuk menyambut Anda di Open Source :) Kami dengan senang hati memandu Anda melalui prosesnya.

Kami memiliki sistem pengalihan yang menggunakan tag -- baca selengkapnya di https://publiclab.org/wiki/power-tags -- tetapi memerlukan tag untuk menunjuk ke "id simpul" seperti redirect:12345 . Ini sulit diingat dan agak tidak jelas -- jadi mari kita gunakan halaman "slug" juga. Kemudian Anda dapat mengarahkan ulang ke halaman di /wiki/openhour dengan menggunakan redirect:openhour .

Langkah demi Langkah

• [ ] Klaim masalah ini : Komentar di bawah. Jika orang lain telah mengklaimnya, tanyakan apakah mereka sudah membuka permintaan tarik dan apakah mereka buntu -- mungkin Anda dapat membantu mereka memecahkan masalah atau meneruskannya!

• [ ] Perbarui file app/controllers/wiki_controller.rb di plots2 (tekan ikon pena kecil) dan edit baris seperti yang ditunjukkan di bawah ini.

Lihat halaman ini untuk mendapatkan bantuan dalam mengambil langkah pertama Anda!

Di bawah ini adalah "diff" yang ditampilkan dalam warna merah (dan - ) baris mana yang harus dihapus, dan dalam warna hijau (dan + ) baris mana yang akan ditambahkan:

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

• [] 💾 Komit perubahan Anda

• [ ] Mulai Permintaan Tarik . Ada dua cara untuk memulai permintaan tarik:

1. Jika Anda familiar dengan terminal atau ingin mempelajarinya, berikut adalah tutorial yang bagus tentang cara mengirim pull request menggunakan terminal.

2. Anda juga dapat mengedit file langsung di browser Anda dan membuka permintaan tarik dari sana.

• [ ] Selesai Tanya di komentar untuk review :)

Pertanyaan?

Tinggalkan komentar di bawah!

Apakah orang lain sudah mengerjakan ini?

Kami mendorong Anda untuk menautkan ke masalah ini dengan menyebutkan masalah # dalam permintaan tarik Anda, sehingga kami dapat melihat apakah seseorang sudah memulainya. Jika seseorang tampak buntu, tawarkan bantuan! Jika tidak, lihat beberapa masalah lain yang dapat Anda bantu . Terima kasih!

(Masalah ini dibuat oleh First-Timers-Bot .)