Plots2: Рефакторинг кода profile.html.erb, часть 2

Первый раз?

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

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

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

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

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

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

Эта проблема

Наш код в app/views/users/profile.html.erb очень подробный ... мы могли бы использовать некоторый рефакторинг

Мы можем начать с рефакторинга повторяющейся строки из кода. На данный момент первая строка:

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

Решение

На что его поменять:

Выполните рефакторинг строки в блоке кода, находящемся в разделе «Проблема», чтобы вместо этого использовать ваш новый вспомогательный метод.

Эту строку можно обновить до <% if current_profile_user %>

Спасибо!!

Шаги по исправлению

• [] Заявите об этой проблеме, добавив комментарий здесь, ниже, и задайте любые уточняющие вопросы, которые вам нужны.
• [] Разветвите репозиторий и настройте его локально, следуя инструкциям README для основного репозитория https://github.com/publiclab/plots2
  
  
  
  • [] Создайте новую ветку функции с уникальным именем, описывающим проблему.
  
  
• [] Попробуйте решить проблему, выполнив описанные выше действия, но даже до того, как вы закончите, вы можете:
  зафиксируйте свои изменения в своей ветке и запустите запрос на перенос (см. участие в программном обеспечении Public Lab ), но отметьте его как «в процессе», если у вас есть вопросы или вы еще не закончили
• [] Укажите эту проблему в теле запроса на перенос.
• [] После отправки запроса на перенос будет предоставлен дополнительный контрольный список для его объединения.

💬 Получить помощь

Если вам нужна помощь - вот несколько вариантов:

• Комментарий ниже
• Присоединяйтесь к нашему gitter-чату на https://gitter.im/publiclab/publiclab