Plots2: Masalah Perbaikan Editor Komentar LAMA TUTUP

Dibuat pada 9 Okt 2020  ·  14Komentar  ·  Sumber: publiclab/plots2

MASALAH PERENCANAAN Pindah DI SINI -> #9069

Saya telah memutuskan untuk memindahkan masalah perencanaan saya ke halaman masalah baru, sebagian besar untuk masalah keterbacaan. Maaf atas kebingungan yang disebabkan oleh hal ini! - @noi5e

outreachy planning

Komentar yang paling membantu

@jywarren Terima kasih banyak untuk tulisan yang sangat membantu dan mendetail!!! Itu sangat membantu saya mengetahui banyak hal tentang basis kode. Masih banyak belajar. Saya membuat PR tentatif untuk ujian-- dengan banyak pertanyaan.

Semua 14 komentar

Untuk tahap selanjutnya dari proyek Editor Komentar ketika mungkin dipintal ke dalam repositorinya sendiri, kami mungkin ingin mempertimbangkan dengan hati-hati alur kerja ini yang didokumentasikan oleh Sagarpreet untuk apa yang akan membantu membuat siklus rilis standar!

https://publiclab.org/notes/sagarpreet_chadha/10-20-2020/first-timer-only-release-workflow

Memperhatikan contoh kotak komentar lintas kabel ini selama unggahan gambar ke komentar yang sedang diedit (bukan yang baru ditulis): https://github.com/publiclab/plots2/issues/8670

MASALAH PERENCANAAN Pindah DI SINI -> #9069

Saya telah memutuskan untuk memindahkan masalah perencanaan saya ke halaman masalah baru, sebagian besar untuk masalah keterbacaan. Maaf atas kebingungan yang disebabkan oleh hal ini! - @noi5e

Hai @noi5e , masalah perencanaan tampak hebat.
Masalah apa yang sedang Anda kerjakan saat ini? Bisakah Anda mengarahkan saya ke PR jika ada, juga beri tahu saya jika Anda terjebak di suatu tempat atau mendapatkan kejelasan. Terima kasih

@sagarpreet-chadha Terima kasih telah menghubungi saya! Sejak memposting ini, telah menulis blogpost saya untuk Outreachy (dalam potongan-potongan, hampir selesai), meneliti dan mengerjakan PR untuk 8618, dan memoles Rails-- yang saya tidak memiliki latar belakang yang kuat, tetapi telah belajar banyak minggu ini. Bekerja 2 hari di pekerjaan saya yang lain jadi saya merasa sedikit ketinggalan, tetapi melakukan banyak hal untuk mengejar hari ini dan akan melakukan lebih banyak besok & Senin. Saya berencana menulis tes untuk komentar besok.

Saya perlu sedikit klarifikasi tentang pengujian komentar. Saya melihat /plots2/blob/main/test/system/comment_test.rb dan sepertinya sudah ada tes untuk memasukkan komentar baik secara manual maupun melalui URL API, keduanya mengarah ke /wiki/wiki-page-path/comments .

Jika saya akan menulis tes komentar baru untuk halaman pertanyaan dan catatan, apakah itu untuk URL API yang berbeda atau sama seperti di atas ( wiki-page-path )? Saya menjalankan rails routes tetapi tidak jelas bagi saya yang mana yang harus saya uji. Terima kasih sebelumnya, ini akan menghemat waktu mencari kode. 😅

EDIT: Beri tahu saya juga jika Anda memiliki permintaan khusus untuk jenis tes untuk saya tulis!

Hai @noi5e saya percaya rute "komentar wiki" berbeda dari "catatan" - wiki dan catatan adalah varian dari Node - tetapi harus sangat mirip.

Sudah lama sejak saya melihat area kode ini, tetapi biasanya Rails akan memiliki tindakan pengontrol untuk setiap rute, tetapi pengecualiannya adalah ketika mereka adalah rute yang banyak akal: https://guides.rubyonrails.org/routing.html#resource -routing-the-rails-default

Melihat tindakan ini adalah untuk melihat komentar:

https://github.com/publiclab/plots2/blob/cbb807ba8e2302f09dafc0060475aa118e34c2c6/app/controllers/wiki_controller.rb#L467 -L470

sepertinya tidak ada tindakan pengontrol untuk memposting. Tetapi saya melihat bahwa rute ini:

https://github.com/publiclab/plots2/blob/cbb807ba8e2302f09dafc0060475aa118e34c2c6/config/routes.rb#L58

menunjuk pada tindakan tampilan itu. Jadi di mana rute dan tindakan pembuatan? Saya melihat beberapa di sini, bukan khusus wiki:

https://github.com/publiclab/plots2/blob/cbb807ba8e2302f09dafc0060475aa118e34c2c6/config/routes.rb#L368 -L373

Jadi mari kita lihat ujung lainnya, template. Berikut adalah formulir komentar dasar:

https://github.com/publiclab/plots2/blob/main/app/views/comments/_form.html.erb

Saya pikir komentar pertanyaan dapat menggunakan kembali formulir ini, berdasarkan logika dalam tag <form> sini?

https://github.com/publiclab/plots2/blob/cbb807ba8e2302f09dafc0060475aa118e34c2c6/app/views/comments/_form.html.erb#L3

Jadi, mereka menunjuk /comments/create/NID dengan metode POST, jadi rute terakhir dari jalur 373 dari route.rb. Titik-titik itu pada tindakan pengontrol komentar create :

https://github.com/publiclab/plots2/blob/cbb807ba8e2302f09dafc0060475aa118e34c2c6/app/controllers/comment_controller.rb#L19 -L56

Jadi saya pikir aman untuk mengatakan bahwa sebagian besar komentar menggunakan tindakan pengontrol komentar create . Tetapi bagaimana dengan pengujian sistem di mana rute itu tidak disebutkan?

Cara lain yang terjadi di sini adalah melalui metode JavaScript:

https://github.com/publiclab/plots2/blob/cbb807ba8e2302f09dafc0060475aa118e34c2c6/test/system/comment_test.rb#L32

Itu membawa kita ke file JS ini, yang mengirimkannya ke rute yang sama melalui AJAX:

https://github.com/publiclab/plots2/blob/484bf69ead7a1f69c9047a25b3f63b9d6f875728/app/assets/javascripts/comment.js#L62 -L70

Cara basis kode kita terlihat sekarang, fungsi sendFormSubmissionAjax() mungkin terlihat seperti tidak harus dalam filenya sendiri, tapi menurut saya @nstjean (alum Outreachy yang fantastis, hai!!!!!! ) berencana untuk merampingkan pengiriman formulir JS di seluruh basis kode kami, yang saya dukung sepenuhnya. Ini dia!

https://github.com/publiclab/plots2/blob/876d0fc084064aaecc23f8003630d7d1ab858fa1/app/assets/javascripts/submit_form_ajax.js#L1 -L4

Semoga ini bermanfaat! Ini bukan yang terburuk yang pernah saya lihat dalam hal organisasi kode, tapi itu pasti berbelit-belit. Ini sebagian hanya karena ini adalah basis kode yang besar dan lama yang telah mengalami banyak fase revisi yang berbeda. Sebenarnya, sebelum pengujian sistem, yang baru diinstal dalam 2 tahun terakhir atau lebih, kami tidak memiliki cara untuk melakukan pengujian tumpukan penuh dari pengiriman komentar JavaScript! Jadi itu terus-menerus melanggar :-(

Terima kasih orang-orang!!!

@jywarren Terima kasih banyak untuk tulisan yang sangat membantu dan mendetail!!! Itu sangat membantu saya mengetahui banyak hal tentang basis kode. Masih banyak belajar. Saya membuat PR tentatif untuk ujian-- dengan banyak pertanyaan.

Juga ingin mengklarifikasi ini:

"Simpan dan pulihkan ikon Fitur utama yang menyebabkan Mimi bekerja" (jangan mengerti ini)

Saya sebenarnya tidak dapat menemukan ini di https://pad.publiclab.org/p/outreachy - dari mana asalnya? Kedengarannya samar-samar akrab ... maaf!

Pratinjau terkadang tidak berfungsi (saya tidak dapat mereproduksi)

Mungkin ini adalah kandidat yang baik untuk jenis "pengujian standar" yang dapat kami jalankan secara otomatis pada beberapa varian editor komentar, ketika kami mengatur ulang kode pengujian? Saya ingin tahu apakah itu akan menangkapnya, atau mungkin itu baru saja diselesaikan.

Tombol tebal dan miring adalah "kabel silang" (tidak dapat direproduksi)

Ini sepertinya ada di https://github.com/publiclab/plots2/issues/8478 dan saya ingin tahu apakah strategi yang sama dapat membantu, termasuk memasukkan ini dalam rangkaian standar tes editor komentar dan berjalan di semua varian.

Pastikan "seret dan lepas untuk menyisipkan gambar" abu-abu ada di mana-mana (tidak dapat mereproduksi, ingin beberapa contoh)

Sama seperti di atas mungkin ... dan saya juga tidak dapat menemukannya di https://pad.publiclab.org/p/outreachy jadi mungkin saya bisa menambahkan lebih banyak, mengetahui dari mana asalnya?

Masalah dengan menyukai komentar (Lihat "Komentar Suka Tidak Berfungsi #5113"). Saya mengerti masalahnya, pada dasarnya saya perlu mengklarifikasi apa perilaku yang diharapkan di sini.

Ini mungkin benar-benar tua?? Kami tidak memiliki suka komentar lagi, karena digantikan oleh sistem "reaksi". Tapi, mungkin kita klarifikasi dengan bertanya pada Sasha, siapa yang membuat #5113

Terima kasih, @noi5e ! Semoga ini membantu!!!!

Terima kasih @jywarren sangat membantu ... Saya pikir poin yang Anda cari ada di dokumen Google ini

Hai @ noi5e saya tidak yakin apakah Anda melihat ini atau sudah memperbaikinya, tetapi saya perhatikan ini di komentar kami - sepertinya ada bilah abu-abu yang tidak perlu di bawah formulir:

image

Juga, saya pikir beberapa jarak di sekitar emoji agak kurang. Lihat bagaimana tampilannya memiliki lapisan putih di bagian atas dan bawah, terutama jika dibandingkan dengan gaya GitHub?

image

Hanya beberapa hal kecil yang mungkin bisa kita bahas dalam PR yang akan datang, tapi tidak ada yang mendesak. Terima kasih!

@jywarren Pasti, saya juga memperhatikannya! Menambah to-dos saya.

Saya juga bertanya-tanya setelah optimasi kecepatan https://github.com/publiclab/plots2/pull/9045 , saya ingat bahwa Skylight menunjukkan bahwa sebenarnya memposting komentar di PublicLab.org dapat memakan waktu cukup lama ... dan saya pikir kami telah sedikit dioptimalkan tetapi tidak sebanyak yang kami harapkan. Apakah Anda masih merasa waktu buka yang lambat untuk mengirim komentar? Saya ingin tahu apakah itu juga cara untuk meningkatkan runtime pengujian sistem, sambil meningkatkan pengalaman pengguna juga.

Berikut cuplikannya; ternyata masih membutuhkan waktu hingga 10 detik kadang-kadang, tapi sayangnya pemantauan kami tidak cukup baik untuk menunjukkan lebih dari itu waktu ini dihabiskan di controller, sepertinya:

image

Ini juga bukan persyaratan proyek Anda, tetapi saya pikir ini mungkin menarik. Kode mengemudi ini di sini dan di sini .

Sejujurnya, tidak perlu khawatir tentang ini ... tetapi saya ingin tahu apakah Anda merasa waktu posting komentar cukup lambat atau tidak?

@jywarren Ya! Saya benar-benar berpikir bahwa waktu untuk mengirim komentar sangat lambat! (juga, waktu untuk bereaksi terhadap komentar)

Menarik untuk dipelajari tentang Skylight dan apa fungsinya. Saya ingin tahu apa sebenarnya yang menyebabkan pelambatan, apakah menurut Anda itu mungkin kueri ActiveRecord?

EDIT: menambahkannya ke daftar harapan tujuan dalam Masalah Perencanaan ini!

MASALAH PERENCANAAN Pindah DI SINI -> #9069

Saya telah memutuskan untuk memindahkan masalah perencanaan saya ke halaman masalah baru, sebagian besar untuk masalah keterbacaan. Maaf atas kebingungan yang disebabkan oleh hal ini! - @noi5e

Apakah halaman ini membantu?
0 / 5 - 0 peringkat