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

ユーザー個別ページのコメント一覧のtitleタグを修正 #7865

Merged
merged 1 commit into from
Jun 24, 2024

Conversation

su-su-su-su
Copy link
Contributor

@su-su-su-su su-su-su-su commented Jun 17, 2024

Issue

概要

titelタグ、og:titleのユーザー名がさん付けになるように修正しました。

変更確認方法

  1. feature/correct-title-tag-user-comments-list-pageをローカルに取り込む
  2. foreman start -f Procfile.devでサーバーを立ち上げる
  3. ユーザー(sotugyou等)でログイン後マイプロフィールをクリック
  4. コメントをクリック
  5. titelタグが○○さんのコメント一覧となっていることを確認 (development) はローカル環境時にだけ表示されるものなので無視
  6. og:titleも○○さんのコメント一覧となっていることを確認

Screenshot

変更前

d7588855a2d229ef677f953022463942

変更後

27271c8a22d96255ab6b45270461a7fa

@su-su-su-su su-su-su-su self-assigned this Jun 17, 2024
@su-su-su-su
Copy link
Contributor Author

@nishitatsu-dev
お疲れ様です。
2週間前からチーム開発に加わりましたsu-su-su-suです。
可能でしたらお手隙の際に、こちらのPRのレビューをお願いできますでしょうか?
ご都合が合わない場合はおっしゃってください。
どうぞよろしくお願いいたします。

@nishitatsu-dev
Copy link
Contributor

@su-su-su-su
レビュー承知しました〜
1週間以内に対応しますので、少々お待ちください🙏

@su-su-su-su
Copy link
Contributor Author

ありがとうございます。
よろしくお願いいたします🙇

Copy link
Contributor

@nishitatsu-dev nishitatsu-dev left a comment

Choose a reason for hiding this comment

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

@su-su-su-su
動作とコード確認しました〜。勉強になりました。
LGTMなので、approveします!!!

細かい話ですみませんが、以下の2箇所を修正して先に進んで下さい〜🙏

  • このプルリクのタイトルの、titleのつづり
  • 変更確認方法に書いてあるブランチ名の、commentのつづり

@su-su-su-su su-su-su-su changed the title ユーザー個別ページのコメント一覧のtitelタグを修正 ユーザー個別ページのコメント一覧のtitleタグを修正 Jun 20, 2024
@su-su-su-su
Copy link
Contributor Author

@nishitatsu-dev

お忙しい中ご確認頂きありがとうございます。

気づけていなかった点をご指摘頂いてありがとうございます🙇
確認怠らないようにしたいと思います!
こちら修正いたしました。

@nishitatsu-dev
Copy link
Contributor

@su-su-su-su
ご対応ありがとうございます🙏
komagataさんレビューに進んでいただいてOKです〜

@su-su-su-su
Copy link
Contributor Author

@nishitatsu-dev

ご確認ありがとうございました🙇‍♂️
komagataさんレビュー依頼いたします!

@su-su-su-su
Copy link
Contributor Author

@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 b09dad1 into main Jun 24, 2024
7 checks passed
@komagata komagata deleted the feature/correct-title-tag-user-comments-list-page branch June 24, 2024 17:44
@github-actions github-actions bot mentioned this pull request Jun 24, 2024
13 tasks
@su-su-su-su
Copy link
Contributor Author

@komagata
ご確認ありがとうございます🙇

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