Skip to content

ログインしていないときのトップページの meta description を変更#6637

Merged
komagata merged 4 commits intomainfrom
feature/change-meta-description-when-not-logged-in
Jun 21, 2023
Merged

ログインしていないときのトップページの meta description を変更#6637
komagata merged 4 commits intomainfrom
feature/change-meta-description-when-not-logged-in

Conversation

@sochi419
Copy link
Copy Markdown
Contributor

@sochi419 sochi419 commented Jun 13, 2023

Issue

概要

ログインしていないときのトップページの meta description を変更しました。

変更確認方法

  1. feature/display-username-in-latest-announcementsをローカルに取り込む
  2. bin/rails sでローカル環境を立ち上げる
  3. localhost:3000にアクセス
  4. アクセスして、ログアウト状態である(以下画像のようにログインしていない)ことを確認する。ログインしているならログアウトをする。
image
  1. デベロッパーツールを開く。
  2. meta descriptionが、修正されていることを確認する。
image

Screenshot

変更前

image

変更後

image

@sochi419 sochi419 self-assigned this Jun 13, 2023
@sochi419 sochi419 requested a review from OdenTakashi June 19, 2023 07:30
@sochi419
Copy link
Copy Markdown
Contributor Author

@OdenTakashi

お疲れ様です!

こちらのissueの確認をお願いできますでしょうか?
急ぎではないです〜

@OdenTakashi
Copy link
Copy Markdown
Member

@sochi419
お疲れ様です!
お返事遅れてしまい大変申し訳ありません💦

レビューのご依頼ありがとうございます🙏
こちら確認させていただきます。!

@OdenTakashi
Copy link
Copy Markdown
Member

@sochi419
PRがdraftになっていますが、何か理由があったりしますか?

@sochi419 sochi419 marked this pull request as ready for review June 21, 2023 06:32
@sochi419
Copy link
Copy Markdown
Contributor Author

@OdenTakashi

すみません、ドラフトのままレビュー依頼していました。
ready for reviewの状態にしたので、確認よろしくお願いします🙏

Copy link
Copy Markdown
Member

@OdenTakashi OdenTakashi left a comment

Choose a reason for hiding this comment

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

コード、動作確認完了しました。
問題ないと思うので自分の方からはApproveとさせていただきます🙏

@sochi419
Copy link
Copy Markdown
Contributor Author

@OdenTakashi

確認ありがとうございました🙏

@sochi419 sochi419 requested a review from komagata June 21, 2023 08:09
@sochi419
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 76bf421 into main Jun 21, 2023
@komagata komagata deleted the feature/change-meta-description-when-not-logged-in branch June 21, 2023 14:19
@github-actions github-actions bot mentioned this pull request Jun 21, 2023
13 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