✉️ /news ページを追加してニュース記事一覧を表示 #1774
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- NewsController#indexアクションを実装 - 既存のNewsモデル（ActiveRecord）を使用 - すべてのニュース記事を最新順に表示するビューを作成 - /news ルーティングを追加 - テスト（spec/requests/news_spec.rb）を追加 /docsと同様に、ニュース記事の一覧ページを提供
- ホームページの#newsセクションと同じようにニュースタイトルを表示 - プリセット絵文字の自動追加機能を適用 - テストを更新して絵文字追加に対応
- format_news_title → News#formatted_title に移動 - news_link_url → News#link_url に移動 - ApplicationHelperから News 関連のメソッドを削除 - ビューファイルをモデルメソッドを使用するように更新 - テストも適切な場所（spec/models/news_spec.rb）に移動 Single Responsibility Principleに従い、Newsリソースに関連する ロジックはNewsモデルに配置することでコードの保守性を向上
- shared/_dojo_letter_signup パーシャルを作成（Rails命名規則に従う） - ホームページの#newsセクションでパーシャルを使用 - /newsページでも同じパーシャルを使用 - コードの重複を削除してDRY原則を適用
- 外部サイト(news.coderdojo.jp)ではなく内部の /news ページへリンク - news_url ヘルパーの代わりに news_index_path を使用
- ヘッダーのNews リンク: '/#news' → news_index_path - フッターのNews リンク: '/#news' → '/news' - フッターのNews アイコン: news_url → news_index_path これにより、すべてのNewsリンクが外部サイト(news.coderdojo.jp)ではなく 内部の /news ページを指すように統一されました
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
概要
/docsと同様に、ニュース記事の一覧を表示する/newsページを追加しました。https://coderdojo.jp/news
変更内容
1. /news インデックスページの実装
NewsController#indexアクションを追加Newsモデル（ActiveRecord）を使用してデータベースからニュース記事を取得/newsルーティングを追加2. Fat Helperのリファクタリング
format_news_title→News#formatted_titleに移動news_link_url→News#link_urlに移動3. DojoLetter登録フォームのパーシャル化
shared/_dojo_letter_signupパーシャルを作成（Rails命名規則に従う）テスト
spec/requests/news_spec.rb- 5つのテストケース追加spec/models/news_spec.rb- モデルメソッドのテスト追加確認事項
News.recentスコープで最新順に表示