Skip to content

日報・みんなのブログのtitle要素を設定した#9206

Merged
komagata merged 1 commit intomainfrom
chore/set-title-reports-and-external-entries
Oct 21, 2025
Merged

日報・みんなのブログのtitle要素を設定した#9206
komagata merged 1 commit intomainfrom
chore/set-title-reports-and-external-entries

Conversation

@yokomaru
Copy link
Copy Markdown
Contributor

@yokomaru yokomaru commented Sep 25, 2025

Issue

概要

  • content_for記法が使われていたが、layoutファイルにyieldする箇所がなかったためタイトルが表示されていなかった
  • 他のページと合わせて使用しているgem mete-tagsの記法である- titleに修正した(以前のやり方へ戻した)

変更確認方法

  1. chore/set-title-reports-and-external-entriesをローカルに取り込む
  2. 任意のユーザーでログイン
  3. 日報のページ(http://localhost:3000/reports) にアクセスし、ページタイトルが日報 | FBCとなっていることを確認
  4. みんなのブログのページ(http://localhost:3000/external_entries) にアクセスし、ページタイトルがみんなのブログ | FBCとなっていることを確認

Screenshot

変更前

  • ページタイトルがFBCとなっている

日報

みんなのブログ

変更後

  • 各々適切なページタイトルが表示されている

日報

みんなのブログ

  • ブラウザのタブだと長くて見切れてしまうので開発ツールで確認

Summary by CodeRabbit

  • Refactor
    • 外部エントリ(「みんなのブログ」)と日報一覧ページのページタイトル設定を見直し、タイトル表示を統一・更新しました。ブラウザタブや共有時の表記がより明確になります。
  • Chores
    • 上記ページのメタディスクリプション設定を削除しました。検索結果やプレビューでの説明文は既定値・他設定に依存します。

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Sep 25, 2025

Walkthrough

2つの一覧ビューで、ページタイトルの設定を content_for から title ヘルパーに置換し、meta description の set_meta_tags 呼び出しを削除しました。その他のテンプレート構造は変更していません。

Changes

Cohort / File(s) Summary of Changes
ビューのタイトル/メタ設定の置換
app/views/external_entries/index.html.slim, app/views/reports/index.html.slim
content_for :title, ...title ... に変更し、set_meta_tags description: ... を削除。その他の描画は既存のまま。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related issues

Possibly related PRs

Suggested reviewers

  • komagata

Poem

葉っぱの見出し ひらりと変え
タイトル一声 しゃんとなり
メタの囁き 今日はおやすみ
ぴょんと跳ねれば ページも軽やか
うさぎの筆で すっきり完了 🐇✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title Check ✅ Passed このタイトルは「日報」と「みんなのブログ」でtitle要素を設定した主要な変更を簡潔かつ的確に示しており、変更内容をひと目で理解できる表現になっています。
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
Description Check ✅ Passed このプルリクエストの説明にはIssue番号、概要、変更確認方法、変更前後のスクリーンショットがテンプレートに従って網羅的に記載されており不足している項目はありません。
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/set-title-reports-and-external-entries

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

- content_for記法が使われていたが、layoutファイルにyieldする箇所がなかったためタイトルが表示されていなかった
- 他のページと合わせて使用しているgem mete-tagsの記法である`- title`に修正した(以前のやり方へ戻した)
@yokomaru yokomaru force-pushed the chore/set-title-reports-and-external-entries branch from 56efdca to 2319e51 Compare October 6, 2025 07:52
@yokomaru yokomaru marked this pull request as ready for review October 8, 2025 07:34
@github-actions github-actions Bot requested a review from komagata October 8, 2025 07:34
@yokomaru
Copy link
Copy Markdown
Contributor Author

yokomaru commented Oct 9, 2025

@sharoa119

sharoaさん、お疲れ様です!🙌
以前Issueの方で確認いただいた、日報・みんなのブログのtitle要素の件につきまして、
修正いたしましたので、レビューも一緒にお願いしてもよろしいでしょうかっ・・・?
全く急ぎではないので、お手隙で&可能でしたらお願いいたします!🙏

@sharoa119 sharoa119 self-requested a review October 10, 2025 01:21
@sharoa119
Copy link
Copy Markdown
Contributor

sharoa119 commented Oct 10, 2025

@yokomaru さん
おはようございます!
レビュー依頼ありがとうございます☺️
お引き受けいたします、と言いますか、先ほど確認させていただきました😀

問題ないと思いますので、approveいたします✨

よろしくお願いいたします🙇‍♀️

@yokomaru
Copy link
Copy Markdown
Contributor Author

@sharoa119
sharoaさん、お返事遅くなり大変申し訳ありません!🙇‍♀️
レビュー引き受けてくださり&爆速でご確認いただきありがとうございましたっ🚀
引き続き、よろしくお願いいたします🙌

@yokomaru
Copy link
Copy Markdown
Contributor Author

@komagata
メンバーレビューが終わりました!
お手隙でレビューをお願いします🙏

Copy link
Copy Markdown
Member

@komagata komagata left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

確認させていただきました。OKです〜!

@komagata komagata merged commit bde6a47 into main Oct 21, 2025
15 checks passed
@komagata komagata deleted the chore/set-title-reports-and-external-entries branch October 21, 2025 15:22
@github-actions github-actions Bot mentioned this pull request Oct 21, 2025
16 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants