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

Add pages metadata #1093

Open
wants to merge 6 commits into
base: nextjs
Choose a base branch
from
Open

Add pages metadata #1093

wants to merge 6 commits into from

Conversation

K-Markopoulos
Copy link
Collaborator

@K-Markopoulos K-Markopoulos commented Feb 10, 2025

(Waiting for NextJS migration to me merged first)

Add metadata content to enhance preview links shared on social media and messages - closes #641

  • Content is based on Added title & meta description + image for all plages #1050
  • Some sites have name: null and we fallback to Site ${id}. Feel free to suggest otherwise.
  • Env var NEXT_PUBLIC_WEBSITE_BASE_URL is now required at build time, to generate absolute URLs for images
  • We use /sites api on each page generation because of caching on the same request.

Example:
image

Copy link

github-actions bot commented Feb 10, 2025

Build succeeded and deployed at https://aqualink-app-1093.surge.sh
(hash 4bf6687 deployed at 2025-02-13T12:53:18)

@Caesarh97
Copy link
Collaborator

Caesarh97 commented Feb 11, 2025

Hi @K-Markopoulos,
Good job on this! Just FYI, the main reason for doing this is from a search engine perspective. Since many of our pages don't have much text, it's hard to gain organic and paid searches for our pages. The title and meta description should help the search engine understand what's on our pages and who should visit them. If you have any other thoughts that could help me with my marketing efforts, please let me know.

Everything looks good to me. Can you confirm if the title and meta for the start page (landing) also have been added? UPDATE: I found the correct info, and it all looks good.

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.

2 participants