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

卒業生の声ページのheadタグを正しいものに修正した #8228

Closed
wants to merge 2 commits into from

Conversation

mousu-a
Copy link
Contributor

@mousu-a mousu-a commented Nov 28, 2024

ブランチ名を間違えて作成してしまったので新しくプルリクを作り直しました🙇‍♂️

新しいプルリク

Issue

概要

/alumni_voicesのtitle, og:title, twitter:title, description, og:description, twitter:description を正しいものに修正しました。

変更確認方法

  1. chore/fix-alumni-voices-head-tagをローカルに取り込む
    1. git fetch origin chore/fix-alumni-voices-head-tag
    2. git checkout chore/fix-alumni-voices-head-tag
  2. 卒業生の声ページに飛ぶ
  3. 開発者ツールでtitle, og:title, twitter:title, description, og:description, twitter:descriptionが卒業生の声のページのものになっていることを確認 (title descriptionでそれぞれ検索すると簡単です)

Screenshot

変更前

FAQのものになってしまっている
image

変更後

卒業生の声のページに相応しいものになっている
貼り付けた画像_2024_11_28_22_25

@mousu-a mousu-a self-assigned this Nov 28, 2024
@mousu-a mousu-a marked this pull request as ready for review November 28, 2024 14:51
@mousu-a
Copy link
Contributor Author

mousu-a commented Nov 29, 2024

すみません、bugのissueのブランチ名をbug/ではなくchore/で作ってしまい、すでにプルリクを作成してしまっているのですが、この場合ブランチをリネーム(プルリクをクローズして新しく作成)したほうがいいでしょうか?
それともこのままレビューに回してしまって良いでしょうか?
ご教示いただければと思います🙇‍♂️

@machida
Copy link
Member

machida commented Nov 29, 2024

@mousu-a 今回はそのままでもいいのですが、今後業務でpush後にリポジトリ名を変える必要があるシチュエーションに出くわすことがある可能性は高いので、その練習として今回もPRの作り直しをお願いしたいですー💪

@mousu-a
Copy link
Contributor Author

mousu-a commented Nov 29, 2024

@machida
承知いたしました!ありがとうございます!

@mousu-a mousu-a closed this Nov 29, 2024
@mousu-a mousu-a deleted the chore/fix-alumni-voices-head-tag branch November 29, 2024 01:28
@github-actions github-actions bot mentioned this pull request Dec 18, 2024
19 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.

2 participants