fix(seo): JSON-LD structured data and sitemap hostname#423
Merged
gabitoesmiapodo merged 4 commits intofeat/ai-integrationfrom Mar 30, 2026
Merged
fix(seo): JSON-LD structured data and sitemap hostname#423gabitoesmiapodo merged 4 commits intofeat/ai-integrationfrom
gabitoesmiapodo merged 4 commits intofeat/ai-integrationfrom
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
There was a problem hiding this comment.
Pull request overview
Adds SEO-related structured data and makes sitemap generation use a configurable base URL so deployments can emit correct absolute links.
Changes:
- Configure
vite-plugin-sitemaphostname viaPUBLIC_APP_URL(with a fallback). - Add
OrganizationandWebApplicationJSON-LD structured data toindex.html.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
vite.config.ts |
Switch sitemap hostname to be driven by PUBLIC_APP_URL instead of a hardcoded domain. |
index.html |
Embed JSON-LD schema definitions for richer search engine metadata. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
5 tasks
gabitoesmiapodo
force-pushed
the
fix/performance
branch
from
1664165 to
bd51b27
Compare
gabitoesmiapodo
force-pushed
the
fix/seo
branch
from
87c9de1 to
080d617
Compare
gabitoesmiapodo
force-pushed
the
fix/performance
branch
from
bd51b27 to
2332bcb
Compare
gabitoesmiapodo
force-pushed
the
fix/seo
branch
from
080d617 to
5957e47
Compare
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
gabitoesmiapodo
force-pushed
the
fix/performance
branch
from
2332bcb to
b1e43d7
Compare
gabitoesmiapodo
force-pushed
the
fix/seo
branch
from
5957e47 to
e8d3651
Compare
- Add Organization and WebApplication JSON-LD schemas to index.html - Make sitemap hostname configurable via PUBLIC_APP_URL env var with fallback to demo.dappbooster.dev
?? only falls back on null/undefined; an empty string (the default in .env.example) would pass through as the hostname and produce an invalid sitemap. || also handles empty strings.
gabitoesmiapodo
force-pushed
the
fix/performance
branch
from
b1e43d7 to
ba8c91b
Compare
gabitoesmiapodo
force-pushed
the
fix/seo
branch
from
e8d3651 to
d0c83c8
Compare
fernandomg
approved these changes
Mar 26, 2026
Collaborator
Author
|
@fernandomg neither did I... hence my repeated answer to whoever complains about our shitty SEO: "There are people who work exclusively on this." |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
OrganizationandWebApplicationJSON-LD schemas toindex.htmlPUBLIC_APP_URLenv var (fallback:demo.dappbooster.dev)Test plan
pnpm testpasses