Retire /people/:id/all_comments now that the combined feed replaces it
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 68/100
調査の方向性
config/routes.rb、PeopleController、一覧にある all_comments ビューから始め、続いて spec/requests/people_all_comments_spec.rb と config/features.yml の機能エントリを読みます。インライン編集を維持する必要があるか確認し、その後、/comments で引き続き使用されている共有 comments partials を削除せずに、非推奨の route とその呼び出し元が削除または付け替えられていることを確認します。
索引モデルが issue の本文から書いたものです。
説明
🤖 From Claude: Filed as a follow-up to #2355.
What
/people/:id/all_comments is no longer linked from anywhere. #2355 added /people/:id/comments_and_communications — a superset that shows the same aggregated comments interleaved with the person's communications — and repointed every caller at it:
- the combined comments & communications section on the person, event registration, scholarship, story, and story idea forms
topic_subscriptions/_form.html.erbcontinuing_education_registrations/edit.html.erbusers/_form.html.erb
The old page is still reachable by typing the URL, but nothing navigates there.
Why it wasn't done in #2355
The instruction was to repoint the links, not to delete the page — and the two pages aren't quite identical, so retiring it is a judgement call worth making deliberately:
- The new page covers it. Both use
PersonCommentAggregator, both have a composer with the same record picker, and the new one adds communications, more filters, and full-height bodies. - What differs.
all_commentsedits a comment inline (comments/_aggregated_comment+comments/create.turbo_stream.erbprepending into the feed). The combined page is read-only per row — you click through to the record to edit. If inline editing matters, port it before deleting.
Scope if we retire it
config/routes.rb— theget :all_commentsmember routePeopleController#all_commentsand its entry in theset_personbefore_actionapp/views/people/all_comments.html.erb,app/views/people/person_comments_results.html.erbspec/requests/people_all_comments_spec.rb- the "Aggregated comments for a person" entry in
config/features.yml(or repoint itsaction_pathat the new page) - check whether
comments/_feedandcomments/_aggregated_commentstill have callers — the global/commentsindex uses them, so they stay
Alternative
Keep it addressable and leave it unlinked. Cheap, but it becomes a second person-comments page that drifts.
- 主要言語
- Ruby
- スター
- 15
- フォーク
- 28
- 平均マージ
- 1日 12分
- マージ済み PR（30日）
- 262
環境構築
このプロジェクトの開発コンテナを、あなたの GitHub アカウントでブラウザ上に起動します。
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
rubyforgood/awbw のほかの issue
-
blocked
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
rubyforgood/awbw#2824 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
rubyforgood/awbw#2822 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
rubyforgood/awbw#2160 ·
メンテナーはふだん 1 日以内に返信
-
Update callout faq defualt text when the new person subscription launches対応中かも @maebeale が 80 日前に担当しました。 オープンblocked
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
rubyforgood/awbw#2095 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
rubyforgood/awbw#2088 ·
メンテナーはふだん 1 日以内に返信
rubyforgood/awbw の issue をすべて見る
似ている issue
-
Missing de translation: components/common/CommunitySolution.tsx:communitySolution.passesLatestTestsオープン
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1時間未満 初心者へのやさしさ 63/100
hitobito/hitobito_bdp#58 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
メンテナーはふだん 10 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
メンテナーはふだん 2 日以内に返信
-
area: devops area: integration help wanted lane: free-agent roadmap size: S type: fixture
難易度 2/5 1〜3時間 初心者へのやさしさ 87/100
arminazimi/evergreen#1505 ·
メンテナーはふだん 1 日以内に返信