-
Notifications
You must be signed in to change notification settings - Fork 53
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
feat(adapter-*): update page snippets with meta image #1526
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will do a quick QA for Next and Svelte and approve after
import { repositoryName } from \\"$lib/prismicio\\"; | ||
</script> | ||
|
||
<svelte:head> | ||
<title>{$page?.data.title}</title> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
❓ No check that title is defined both title
and meta
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it's just page
that might be undefined
and title
tag is mandatory-ish. Not sure if we want to do that check for the og:title
meta 🤔
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
QA done for Next, Nuxt and Sveltekit! All good with the latest commit! 🚀
useSeoMeta()
Resolves: nuxt-modules/prismic#222
Description
This pull request updates Nuxt pages snippets to use the modern equivalent of
useHead
:useSeoMeta
, also taking into account meta image.Checklist
Preview
N/A
How to QA 1
Try the new page snippet with:
Footnotes
Please use these labels when submitting a review:
⚠️ #issue: Strongly suggest a change.
❓ #ask: Ask a question.
💡 #idea: Suggest an idea.
🎉 #nice: Share a compliment. ↩