Conversation
Walkthrough2つの一覧ビューで、ページタイトルの設定を content_for から title ヘルパーに置換し、meta description の set_meta_tags 呼び出しを削除しました。その他のテンプレート構造は変更していません。 Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related issues
Possibly related PRs
Suggested reviewers
Poem
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
- content_for記法が使われていたが、layoutファイルにyieldする箇所がなかったためタイトルが表示されていなかった - 他のページと合わせて使用しているgem mete-tagsの記法である`- title`に修正した(以前のやり方へ戻した)
56efdca to
2319e51
Compare
|
sharoaさん、お疲れ様です!🙌 |
|
@yokomaru さん 問題ないと思いますので、approveいたします✨ よろしくお願いいたします🙇♀️ |
|
@sharoa119 |
|
@komagata |
Issue
概要
gem mete-tagsの記法である- titleに修正した(以前のやり方へ戻した)変更確認方法
chore/set-title-reports-and-external-entriesをローカルに取り込むScreenshot
変更前
日報
みんなのブログ
変更後
日報
みんなのブログ
Summary by CodeRabbit