Replies: 1 comment
-
|
I went ahead and submitted a PR: |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
Hey Bryant, is there a reason you're using
NUXT_PUBLIC_SITE_URLinstead ofSITE_URLinnuxt.config.ts?https://github.com/directus-labs/agency-os/blob/main/nuxt.config.ts#L48 - this one in particular
https://github.com/directus-labs/agency-os/blob/main/nuxt.config.ts#L56
https://github.com/directus-labs/agency-os/blob/main/nuxt.config.ts#L100
We ran into an issue with the og:image meta wasn't working properly, since this env var wasn't set in the .env file; it wasn't mentioned in the README here; only the
SITE_URLenv var is mentioned.You can see the discrepancy on your own site as well:
Looks like
NUXT_PUBLIC_SITE_URLis being passed to the Vercel deploy button in the README, but NUXT_PUBLIC_SITE_URL is more of an override for Nuxt SEO - OG Image Installation.With that, I guess the real question is, should this be changed in code to use
SITE_URLinstead ofNUXT_PUBLIC_SITE_URLor should we addNUXT_PUBLIC_SITE_URLto the list of required env vars and leave the code as is?Happy to send over a PR after I better understand what's happening here.
Beta Was this translation helpful? Give feedback.
All reactions