Plots2: Refactoring profile.html.erb kode pt.2

Dibuat pada 19 Des 2018  ·  4Komentar  ·  Sumber: publiclab/plots2

Pertama kali?

Ini adalah masalah pertama kali saja . 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 melihat melalui bantuan umum kami masalah yang

Apa yang perlu Anda ketahui.

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

Masalah

Kode kita di app/views/users/profile.html.erb sangat verbose... kita bisa menggunakan beberapa refactoring

Kita bisa mulai dengan memfaktorkan ulang baris yang muncul kembali dari kode. Untuk saat ini, baris pertama dari:

https://github.com/publiclab/plots2/blob/1931439c0c799b3e3420a9dcaace7a54cf895bd4/app/views/users/profile.html.erb#L216 -L223

Larutan

Apa yang harus diubah menjadi:

Refactor baris di blok kode yang ditemukan di bagian "Masalah" untuk menggunakan metode pembantu baru Anda sebagai gantinya.

Baris itu dapat diperbarui menjadi <% if current_profile_user %>

Terima kasih!!

Langkah-langkah untuk Memperbaiki

  • [ ] Klaim masalah ini dengan komentar di sini, di bawah, dan ajukan pertanyaan klarifikasi yang Anda butuhkan
  • [ ] Fork repositori dan atur secara lokal dengan mengikuti instruksi README repo utama https://github.com/publiclab/plots2

    • [ ] Buat cabang fitur baru dengan nama unik yang menjelaskan masalah

  • [ ] Coba perbaiki masalah dengan mengikuti langkah-langkah di atas, tetapi bahkan sebelum Anda selesai, Anda dapat:
    komit perubahan Anda ke cabang Anda dan mulai permintaan tarik (lihat berkontribusi ke perangkat lunak Lab Publik ) tetapi tandai sebagai "sedang berlangsung" jika Anda memiliki pertanyaan atau jika Anda belum selesai
  • [ ] Referensikan masalah ini di badan permintaan tarik Anda
  • [ ] Setelah Anda mengirimkan permintaan tarik, daftar periksa tambahan akan disediakan untuk menggabungkannya

Dapatkan bantuan

Jika Anda memerlukan bantuan - berikut adalah beberapa opsi:

first-timers-only help wanted

Komentar yang paling membantu

Saya belum membuat kontribusi opensource jadi ini akan menghangatkan saya. Bolehkah saya mengklaim ini?

Semua 4 komentar

Saya belum membuat kontribusi opensource jadi ini akan menghangatkan saya. Bolehkah saya mengklaim ini?

Hai @dave-light, silakan lanjutkan. Dan, Selamat datang di Lab Publik dan Dunia Sumber Terbuka :globe_with_meridians: :earth_americas: .

@dave-light: apakah Anda di dalamnya? atau dapat mengklaim ini?

Hai @mzmmlds , saya ikut.

Apakah halaman ini membantu?
0 / 5 - 0 peringkat