-
Notifications
You must be signed in to change notification settings - Fork 86
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
Fix #103 No Open Graph Meta Tags Present #113
Conversation
@Riteshyadavanshi is attempting to deploy a commit to the Stack Team on Vercel. A member of the Team first needs to authorize it. |
@Riteshyadavanshi bro, where is that issue, can you please show me, like a screen short etc |
@@ -18,11 +18,15 @@ import { RouterProvider } from '@/components/router'; | |||
import { CSPostHogProvider, UserIdentity } from './providers'; | |||
|
|||
export const metadata: Metadata = { | |||
metadataBase:new URL("https://stack-auth.com/"), |
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.
The base url should use the DASHBOARD_BASE_URL
env var
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.
also please make sure the formatting is aligned with the other part of the project
title: { | ||
default: 'Stack Auth Dashboard', | ||
template: '%s | Stack Auth', | ||
}, | ||
description: 'Stack Auth is the fastest way to add authentication to your web app.', | ||
openGraph:{ |
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.
same here
Hey sir ,I have made the changes just now. you can check it
…On Mon, Jul 1, 2024, 10:24 AM Zai Shi ***@***.***> wrote:
***@***.**** commented on this pull request.
------------------------------
In apps/dashboard/src/app/layout.tsx
<#113 (comment)>:
> @@ -18,11 +18,15 @@ import { RouterProvider } from '@/components/router';
import { CSPostHogProvider, UserIdentity } from './providers';
export const metadata: Metadata = {
+ metadataBase:new URL("https://stack-auth.com/"),
The base url should use the DASHBOARD_BASE_URL env var
------------------------------
In apps/dashboard/src/app/layout.tsx
<#113 (comment)>:
> title: {
default: 'Stack Auth Dashboard',
template: '%s | Stack Auth',
},
description: 'Stack Auth is the fastest way to add authentication to your web app.',
+ openGraph:{
same here
------------------------------
In apps/dashboard/src/app/layout.tsx
<#113 (comment)>:
> @@ -18,11 +18,15 @@ import { RouterProvider } from '@/components/router';
import { CSPostHogProvider, UserIdentity } from './providers';
export const metadata: Metadata = {
+ metadataBase:new URL("https://stack-auth.com/"),
also please make sure the formatting is aligned with the other part of the
project
—
Reply to this email directly, view it on GitHub
<#113 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/A444J5FVE65EALXJSFZPN3TZKDOJRAVCNFSM6AAAAABKECUQF6VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDCNJQGM4TOMJSGY>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Please do not open new PRs multiple times. Just update the PR. |
No description provided.