Plots2: Невозможно отказаться от подписки после нажатия следующей кнопки тега.

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

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

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

Проблема

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

Что должно произойти:
Если пользователь уже подписан на тему, повторное нажатие этой кнопки должно отменить подписку пользователя на эту тему.

Что на самом деле происходит:
Пользователь просто перенаправляется на страницу с определенной темой.

Решение

Где найти соответствующие строки кода:
https://github.com/publiclab/plots2/blob/master/app/views/home/subscriptions.html.erb#L9
https://github.com/publiclab/plots2/blob/master/app/views/tag/show.html.erb#L47

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

Нам нужно добавить data-method="delete" к упомянутым выше двум строкам.

<a rel="tooltip" title="<%= t('home.subscriptions.click_unfollow') %>" class="btn btn-default active" href="/unsubscribe/tag/everything"><i class="fa fa-eye"></i> <%= t('home.subscriptions.following') %> <b><%= t('home.subscriptions.research_notes') %></b></a>
к
<a rel="tooltip" title="<%= t('home.subscriptions.click_unfollow') %>" class="btn btn-default active" href="/unsubscribe/tag/everything" data-method="delete"><i class="fa fa-eye"></i> <%= t('home.subscriptions.following') %> <b><%= t('home.subscriptions.research_notes') %></b></a>

а также

<a rel="tooltip" title="<%= t('home.subscriptions.click_unfollow') %>" class="btn btn-default active" href="/unsubscribe/tag/everything"><i class="fa fa-eye"></i> <%= t('home.subscriptions.following') %> <b><%= t('home.subscriptions.research_notes') %></b></a>
к
<a rel="tooltip" title="<%= t('home.subscriptions.click_unfollow') %>" class="btn btn-default active" href="/unsubscribe/tag/everything" data-method="delete"><i class="fa fa-eye"></i> <%= t('home.subscriptions.following') %> <b><%= t('home.subscriptions.research_notes') %></b></a>

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

• [] заявите об этой проблеме, оставив комментарий здесь, ниже, и задайте любые уточняющие вопросы, которые вам нужны.
• [] настройте репозиторий локально, следуя инструкциям README, и убедитесь, что все тесты проходят
• [] попробуйте решить проблему, выполнив описанные выше действия, но даже до того, как вы закончите, вы можете:
• [] зафиксируйте изменения и запустите запрос на перенос (см. участие в программном обеспечении Public Lab ), но отметьте его как «в процессе», если у вас есть вопросы или вы еще не закончили
• [] предупредить кого-нибудь через список разработчиков (см. ниже) о том, что ваш пул-реквест будет объединен. Это может потребовать дополнительных действий в зависимости от того, когда вы это сделаете, но мы вам поможем!

Отправьте электронное письмо списку разработчиков (см. Https://publiclab.org/wiki/developers) или перейдите в чат, если у вас есть вопросы, и посмотрите нашу целевую страницу для новичков для получения дополнительной информации !