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 が反映されるように設定 #12

Merged
merged 4 commits into from
Apr 6, 2023

Conversation

yamanoku
Copy link
Contributor

@yamanoku yamanoku commented Apr 6, 2023

変更前 変更後
「Vue3でtsx firstができるか検証する」記事の title 要素が「ushironoko.me」となっている 「Vue3でtsx firstができるか検証する」記事の title 要素が「Vue3でtsx firstができるか検証する」となっている

@vercel
Copy link

vercel bot commented Apr 6, 2023

Someone is attempting to deploy a commit to a Personal Account owned by @ushironoko on Vercel.

@ushironoko first needs to authorize it.

@@ -25,7 +28,7 @@ useHead({
},
{
name: 'twitter:title',
content: data.value?.title ?? 'ushironoko.me',
content: postTitle ?? 'ushironoko.me',
Copy link
Contributor Author

Choose a reason for hiding this comment

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

同等の値があったので併せて変数にしておきました

Copy link
Contributor

Choose a reason for hiding this comment

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

これだったらpostTitle自体を data.value?.contentHtml ?? 'ushironoko.me' にしたほうがよくないですか?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

contentHtml => title ですかね。たしかに。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

64f65c7 にて変更

Copy link
Contributor

Choose a reason for hiding this comment

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

でもsite_nameはushironoko.meだなー(元々の設定がわるい)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

あ、ほんとだ

Copy link
Contributor Author

Choose a reason for hiding this comment

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

b5510c2 で site_name だけ ushironoko.me にしました

@vercel
Copy link

vercel bot commented Apr 6, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
blog ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 6, 2023 3:07am

Copy link
Owner

@ushironoko ushironoko left a comment

Choose a reason for hiding this comment

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

LGTM

@ushironoko ushironoko merged commit 370ee3e into ushironoko:master Apr 6, 2023
@yamanoku yamanoku deleted the add-post-page-title branch April 6, 2023 03:29
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.

None yet

3 participants