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

add unique ogp #123

Merged
merged 1 commit into from
Jun 26, 2023
Merged

add unique ogp #123

merged 1 commit into from
Jun 26, 2023

Conversation

daimoriwaki
Copy link
Collaborator

#119

実現したいこと

  • URL付きでtweet等をした時に、レビューの概要がOGPとして表示されるようにしたい
  • 今茂木さんがebpmdb広報アカウントでやっていることをスクショ不要でやれるようにしたい

やったこと

  • public/screenshots/ 以下に今上がっているレビューの全スクショを入れました。ややずれているものもありますが、そういうのは後で入れ替えます。
  • [slug].tsxにおいてdocnumberを取得してHeader.tsxに渡すようにしました。
  • Header.tsxにおいてdocnumberを渡されたら対応するスクショをOGPに設定するようにしました。

@daimoriwaki daimoriwaki requested review from ken-g0, kigei and RyoMogi June 24, 2023 14:25
Copy link
Contributor

@ken-g0 ken-g0 left a comment

Choose a reason for hiding this comment

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

LGTM! 👍

@daimoriwaki daimoriwaki merged commit 2692766 into main Jun 26, 2023
@daimoriwaki daimoriwaki deleted the feat/unique_ogp branch June 26, 2023 06:50
@daimoriwaki
Copy link
Collaborator Author

run failしたので後でやり直す

@daimoriwaki
Copy link
Collaborator Author

image

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