Delete vestigial MailingListForm; use button_to for subscribe/unsubscribe
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 88/100
調査の方向性
まず app/views/subscriptions/index.html.haml と app/form_models/mailing_list_form.rb を読み、その後 spec/features にある subscriptions ページのカバレッジを確認してください。ビューを変更する前に、既存の Create および Destroy ルートと MailingListsController のアクションを確認してください。両方のボタンが同じように動作し、unsubscribe が引き続き DELETE を使用し、フォームモデルが削除され、関連する feature specs がパスすれば完了です。
索引モデルが issue の本文から書いたものです。
説明
Problem
app/form_models/mailing_list_form.rb is a one-line ActiveModel::Model class (attr_accessor :name) that exists only as an anchor object for two simple_form_for wrappers in app/views/subscriptions/index.html.haml. Those "forms" are just subscribe/unsubscribe buttons — no fields are rendered, and MailingListsController ignores all params (its create/destroy read nothing). TermsAndConditionsForm stays — it has a real acceptance validation and .valid? gate.
Proposed change
- Replace both
simple_form_for @mailing_list, ...wrappers withbutton_to(native Rails: handles POST/method: :delete, CSRF, and accepts the sameclass: 'btn btn-success btn-lg mb-0'). - Delete
app/form_models/mailing_list_form.rb. - No controller or route changes.
Acceptance
- Subscribe/unsubscribe buttons on the subscriptions page behave identically (including
method: :deletefor unsubscribe). MailingListFormno longer exists.spec/featurescovering the subscriptions page pass.
- 主要言語
- Ruby
- スター
- 104
- フォーク
- 205
- 平均マージ
- 1日 2時間
- マージ済み PR（30日）
- 81
環境構築
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
codebar/planner のほかの issue
-
enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
メンテナーはふだん 1 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 56/100
メンテナーはふだん 1 日以内に返信
-
Chapter show pages allocate ~21k+ objects per render for large chapters (18% of app allocations)オープンperformance
難易度 4/5 3〜5日 初心者へのやさしさ 55/100
codebar/planner#2952 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
-
Workshop show pages allocate ~7.3k objects per request with no caching (28% of app allocations)オープンperformance
難易度 4/5 3〜5日 初心者へのやさしさ 52/100
メンテナーはふだん 1 日以内に返信
codebar/planner の issue をすべて見る
似ている issue
-
security
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
メンテナーはふだん 1 日以内に返信
-
OSCON 2016オープンcontent
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
rubyevents/rubyevents#2148 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
we-promise/sure#3838 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
Mend: dependency security vulnerability
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
ManageIQ/manageiq-ui-classic#10341 ·
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
rubocop/ruby-style-guide#984 ·