Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Docの個別ページのtitleタグ及びmetaタグのog:titleはタイトルの頭にDocと表示するように変更 #7034

Merged
merged 2 commits into from
Nov 15, 2023

Conversation

ham-cap
Copy link
Contributor

@ham-cap ham-cap commented Nov 5, 2023

Issue

概要

Docの個別ページでは、titleタグとmetaタグのog:titleの先頭にDocと表示するように変更しました。

変更確認方法

  1. ブランチfeature/change-title-tag-and-og-titleをローカルに取り込む
  2. rails sでローカル環境を立ち上げる
  3. 任意のアカウントでログイン
  4. Docs内の任意のページを開く
  5. デベロッパーツールで確認

該当箇所

変更前

デベロッパーツール

image

変更後

デベロッパーツール

image

@ham-cap ham-cap self-assigned this Nov 5, 2023
@ham-cap ham-cap force-pushed the feature/change-title-tag-and-og-title branch from 2786ac6 to 098005d Compare November 11, 2023 00:30
@ham-cap ham-cap marked this pull request as ready for review November 11, 2023 08:10
@ham-cap ham-cap requested a review from Kassy0220 November 11, 2023 08:16
@ham-cap
Copy link
Contributor Author

ham-cap commented Nov 11, 2023

@Kassy0220
お疲れ様です!
初issueのレビューをお願いできませんでしょうか🙏
ご対応が難しそうであればお知らせください。
よろしくお願いいたします。

@Kassy0220
Copy link
Contributor

@ham-cap
お疲れ様です!
レビュー依頼了解しました、明日中には返信しようと思いますので、少々お待ちください🙏

@ham-cap
Copy link
Contributor Author

ham-cap commented Nov 11, 2023

ありがとうございます🙏
よろしくお願いいたします🙇‍♂️

Copy link
Contributor

@Kassy0220 Kassy0220 left a comment

Choose a reason for hiding this comment

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

@ham-cap
こちら確認しました!
LGTMですのでApproveいたします〜🙌🙌

すみません、PRの内容とは関係ありませんがPRの概要のところで気になったところがありました。参考までに。

  • 見出しの先頭にはDocをつけないことを示すために見出しのスクショを載せておられますが、こちらは無くても大丈夫だと思います。
    • スクショは変更箇所のみ載せてもらうと、変更対象が明確になるかなと思いました(最初見た時にタイトル表示も変更したのかな?と思ってしまいました)

@@ -1,4 +1,4 @@
- title @page.title
- title "Doc #{@page.title}"
Copy link
Contributor

Choose a reason for hiding this comment

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

以下のgemとメソッドによって、titleタグと<meta property="og:title">タグの値の設定を行なっているのですね📝

kpumuk/meta-tags: Search Engine Optimization (SEO) for Ruby on Rails applications.

def default_meta_tags
{
site: 'FBC',
reverse: true,
charset: 'utf-8',
description: '月額29.800円、全機能が使えるお試し期間付き。フィヨルドブートキャンプは現場の即戦力になるためのスキルとプログラミングの楽しさを伝える、現役エンジニアが考える理想のプログラミングスクールの実現に励んでいます。',
viewport: 'width=device-width, initial-scale=1.0',
og: {
title: :title,

@ham-cap
Copy link
Contributor Author

ham-cap commented Nov 12, 2023

@Kassy0220
早々にご対応いただきありがとうございます🙏

  • 見出しの先頭にはDocをつけないことを示すために見出しのスクショを載せておられますが、こちらは無くても大丈夫だと思います。
    • スクショは変更箇所のみ載せてもらうと、変更対象が明確になるかなと思いました(最初見た時にタイトル表示も変更したのかな?と思ってしまいました)

言われてみるとたしかに間違い探しのようになっていて分かりにくかったですね💦
失礼いたしました🙇‍♂️
画像は消しておきます!

@komagata
お疲れ様です!

チームメンバーのApproveをいただきましたのでレビューをお願いできますでしょうか🙏
よろしくお願いいたします🙇‍♂️

Copy link
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 341f86f into main Nov 15, 2023
6 checks passed
@komagata komagata deleted the feature/change-title-tag-and-og-title branch November 15, 2023 07:05
@github-actions github-actions bot mentioned this pull request Nov 15, 2023
8 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