From 608f16f5781fb65cc47f1d367122501b3bcfadb6 Mon Sep 17 00:00:00 2001
From: Deeman <hendriknote@gmail.com>
Date: Tue, 10 Mar 2026 10:53:20 +0100
Subject: [PATCH] =?UTF-8?q?fix(seo):=20pre-GSC=20audit=20=E2=80=94=20sitem?=
 =?UTF-8?q?ap,=20redirects,=20OG=20tags,=20schema=20markup?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- sitemap: replace /market-score with /padelnomics-score, add /opportunity-map,
  remove /billing/pricing (blocked by robots.txt), deduplicate articles query
- app: fix /market-score redirect chain (→ /en/padelnomics-score directly)
- base.html: move default OG tags inside {% block head %} so child overrides
  replace them instead of duplicating
- features, planner, directory: add JSON-LD WebPage + BreadcrumbList schema
- export pages: add meta descriptions and {% block head %} where missing

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 web/src/padelnomics/app.py                    |  2 +-
 .../directory/templates/directory.html        | 22 ++++++++++++++
 .../padelnomics/planner/templates/export.html |  1 +
 .../planner/templates/export_details.html     |  1 +
 .../planner/templates/export_success.html     |  4 +++
 .../planner/templates/export_waitlist.html    |  4 +++
 .../planner/templates/planner.html            | 29 +++++++++++++++++++
 .../public/templates/features.html            | 22 ++++++++++++++
 web/src/padelnomics/sitemap.py                | 13 +++++----
 web/src/padelnomics/templates/base.html       |  9 +++---
 10 files changed, 96 insertions(+), 11 deletions(-)

diff --git a/web/src/padelnomics/app.py b/web/src/padelnomics/app.py
index 9bbd640..5ec5c53 100644
--- a/web/src/padelnomics/app.py
+++ b/web/src/padelnomics/app.py
@@ -401,7 +401,7 @@ def create_app() -> Quart:
 
     @app.route("/market-score")
     async def legacy_market_score():
-        return redirect("/en/market-score", 301)
+        return redirect("/en/padelnomics-score", 301)
 
     # -------------------------------------------------------------------------
     # Blueprint registration
diff --git a/web/src/padelnomics/directory/templates/directory.html b/web/src/padelnomics/directory/templates/directory.html
index bd5dd65..f6a56a1 100644
--- a/web/src/padelnomics/directory/templates/directory.html
+++ b/web/src/padelnomics/directory/templates/directory.html
@@ -6,6 +6,28 @@
 <meta name="description" content="{{ t.dir_page_meta_desc | tformat(count=total_suppliers, countries=total_countries) }}">
 <meta property="og:title" content="{{ t.dir_page_title }} - {{ config.APP_NAME }}">
 <meta property="og:description" content="{{ t.dir_page_og_desc | tformat(count=total_suppliers, countries=total_countries) }}">
+<script type="application/ld+json">
+{
+  "@context": "https://schema.org",
+  "@graph": [
+    {
+      "@type": "WebPage",
+      "name": "{{ t.dir_page_title }} - {{ config.APP_NAME }}",
+      "description": "{{ t.dir_page_meta_desc | tformat(count=total_suppliers, countries=total_countries) }}",
+      "url": "{{ config.BASE_URL }}/{{ lang }}/directory/",
+      "inLanguage": "{{ lang }}",
+      "isPartOf": {"@type": "WebSite", "name": "Padelnomics", "url": "{{ config.BASE_URL }}"}
+    },
+    {
+      "@type": "BreadcrumbList",
+      "itemListElement": [
+        {"@type": "ListItem", "position": 1, "name": "Home", "item": "{{ config.BASE_URL }}/{{ lang }}"},
+        {"@type": "ListItem", "position": 2, "name": "{{ t.dir_page_title }}", "item": "{{ config.BASE_URL }}/{{ lang }}/directory/"}
+      ]
+    }
+  ]
+}
+</script>
 <style>
   :root {
     --dir-green:      #15803D;
diff --git a/web/src/padelnomics/planner/templates/export.html b/web/src/padelnomics/planner/templates/export.html
index f5c2327..1b72a34 100644
--- a/web/src/padelnomics/planner/templates/export.html
+++ b/web/src/padelnomics/planner/templates/export.html
@@ -3,6 +3,7 @@
 {% block paddle %}{% include "_payment_js.html" %}{% endblock %}
 
 {% block head %}
+<meta name="description" content="{{ t.export_title }}">
 <style>
   .exp-wrap { max-width: 640px; margin: 0 auto; padding: 3rem 0; }
   .exp-hero { text-align: center; margin-bottom: 2rem; }
diff --git a/web/src/padelnomics/planner/templates/export_details.html b/web/src/padelnomics/planner/templates/export_details.html
index d0d2ee8..f828d79 100644
--- a/web/src/padelnomics/planner/templates/export_details.html
+++ b/web/src/padelnomics/planner/templates/export_details.html
@@ -2,6 +2,7 @@
 {% block title %}Business Plan Details — {{ config.APP_NAME }}{% endblock %}
 
 {% block head %}
+<meta name="description" content="Business Plan Details — {{ config.APP_NAME }}">
 <style>
   .bp-wrap { max-width: 680px; margin: 0 auto; padding: 3rem 0; }
   .bp-hero { margin-bottom: 2rem; }
diff --git a/web/src/padelnomics/planner/templates/export_success.html b/web/src/padelnomics/planner/templates/export_success.html
index deac913..894912a 100644
--- a/web/src/padelnomics/planner/templates/export_success.html
+++ b/web/src/padelnomics/planner/templates/export_success.html
@@ -1,6 +1,10 @@
 {% extends "base.html" %}
 {% block title %}{{ t.export_success_title }} - {{ config.APP_NAME }}{% endblock %}
 
+{% block head %}
+<meta name="description" content="{{ t.export_success_title }}">
+{% endblock %}
+
 {% block content %}
 <main class="container-page" style="max-width:500px;margin:0 auto;padding:4rem 1rem;text-align:center">
   <div style="font-size:3rem;margin-bottom:1rem">&#10003;</div>
diff --git a/web/src/padelnomics/planner/templates/export_waitlist.html b/web/src/padelnomics/planner/templates/export_waitlist.html
index e5bea77..b48abfd 100644
--- a/web/src/padelnomics/planner/templates/export_waitlist.html
+++ b/web/src/padelnomics/planner/templates/export_waitlist.html
@@ -2,6 +2,10 @@
 
 {% block title %}{{ t.export_waitlist_title }} - {{ config.APP_NAME }}{% endblock %}
 
+{% block head %}
+<meta name="description" content="{{ t.export_waitlist_title }} - {{ config.APP_NAME }}">
+{% endblock %}
+
 {% block content %}
 <main class="container-page py-12">
   <div class="card max-w-md mx-auto mt-8 text-center">
diff --git a/web/src/padelnomics/planner/templates/planner.html b/web/src/padelnomics/planner/templates/planner.html
index 3f0cdc6..408151e 100644
--- a/web/src/padelnomics/planner/templates/planner.html
+++ b/web/src/padelnomics/planner/templates/planner.html
@@ -9,6 +9,35 @@
 <meta property="og:image" content="{{ url_for('static', filename='images/planner-screenshot.png', _external=True) }}">
 <link rel="stylesheet" href="{{ url_for('static', filename='css/planner.css') }}?v={{ v }}">
 <script defer src="https://cdnjs.cloudflare.com/ajax/libs/Chart.js/4.4.1/chart.umd.min.js"></script>
+<script type="application/ld+json">
+{
+  "@context": "https://schema.org",
+  "@graph": [
+    {
+      "@type": "WebPage",
+      "name": "{{ t.planner_page_title }} - {{ config.APP_NAME }}",
+      "description": "{{ t.planner_meta_desc }}",
+      "url": "{{ config.BASE_URL }}/{{ lang }}/planner/",
+      "inLanguage": "{{ lang }}",
+      "isPartOf": {"@type": "WebSite", "name": "Padelnomics", "url": "{{ config.BASE_URL }}"}
+    },
+    {
+      "@type": "BreadcrumbList",
+      "itemListElement": [
+        {"@type": "ListItem", "position": 1, "name": "Home", "item": "{{ config.BASE_URL }}/{{ lang }}"},
+        {"@type": "ListItem", "position": 2, "name": "{{ t.nav_planner }}", "item": "{{ config.BASE_URL }}/{{ lang }}/planner/"}
+      ]
+    },
+    {
+      "@type": "SoftwareApplication",
+      "name": "Padelnomics Padel Court Planner",
+      "applicationCategory": "BusinessApplication",
+      "operatingSystem": "Web",
+      "offers": {"@type": "Offer", "price": "0", "priceCurrency": "EUR"}
+    }
+  ]
+}
+</script>
 {% endblock %}
 
 {% macro slider(name, label, min, max, step, value, tip='') %}
diff --git a/web/src/padelnomics/public/templates/features.html b/web/src/padelnomics/public/templates/features.html
index 4e91cec..8d2fddc 100644
--- a/web/src/padelnomics/public/templates/features.html
+++ b/web/src/padelnomics/public/templates/features.html
@@ -7,6 +7,28 @@
 <meta property="og:title" content="{{ t.features_title_prefix }} | {{ config.APP_NAME }}">
 <meta property="og:description" content="{{ t.features_meta_desc }}">
 <meta property="og:image" content="{{ url_for('static', filename='images/planner-screenshot.png', _external=True) }}">
+<script type="application/ld+json">
+{
+  "@context": "https://schema.org",
+  "@graph": [
+    {
+      "@type": "WebPage",
+      "name": "{{ t.features_title_prefix }} | {{ config.APP_NAME }}",
+      "description": "{{ t.features_meta_desc }}",
+      "url": "{{ config.BASE_URL }}/{{ lang }}/features",
+      "inLanguage": "{{ lang }}",
+      "isPartOf": {"@type": "WebSite", "name": "Padelnomics", "url": "{{ config.BASE_URL }}"}
+    },
+    {
+      "@type": "BreadcrumbList",
+      "itemListElement": [
+        {"@type": "ListItem", "position": 1, "name": "Home", "item": "{{ config.BASE_URL }}/{{ lang }}"},
+        {"@type": "ListItem", "position": 2, "name": "{{ t.features_title_prefix }}", "item": "{{ config.BASE_URL }}/{{ lang }}/features"}
+      ]
+    }
+  ]
+}
+</script>
 {% endblock %}
 
 {% block content %}
diff --git a/web/src/padelnomics/sitemap.py b/web/src/padelnomics/sitemap.py
index e98f78b..3d27dc8 100644
--- a/web/src/padelnomics/sitemap.py
+++ b/web/src/padelnomics/sitemap.py
@@ -26,9 +26,10 @@ STATIC_PATHS = [
     "/imprint",
     "/suppliers",
     "/markets",
-    "/market-score",
+    "/padelnomics-score",
     "/planner/",
     "/directory/",
+    "/opportunity-map",
 ]
 
 
@@ -65,16 +66,16 @@ async def _generate_sitemap_xml(base_url: str) -> str:
         for lang in LANGS:
             entries.append(_url_entry(f"{base}/{lang}{path}", alternates))
 
-    # Billing pricing — no lang prefix, no hreflang
-    entries.append(_url_entry(f"{base}/billing/pricing", []))
-
     # Published articles — both lang variants with accurate lastmod.
     # Exclude noindex articles (thin data) to keep sitemap signal-dense.
+    # GROUP BY url_path: articles table has one row per language per url_path,
+    # but the for-lang loop already creates both lang entries per path.
     articles = await fetch_all(
-        """SELECT url_path, COALESCE(updated_at, published_at) AS lastmod
+        """SELECT url_path, MAX(COALESCE(updated_at, published_at)) AS lastmod
            FROM articles
            WHERE status = 'published' AND noindex = 0 AND published_at <= datetime('now')
-           ORDER BY published_at DESC
+           GROUP BY url_path
+           ORDER BY MAX(published_at) DESC
            LIMIT 25000"""
     )
     for article in articles:
diff --git a/web/src/padelnomics/templates/base.html b/web/src/padelnomics/templates/base.html
index 3b8b542..8e5905c 100644
--- a/web/src/padelnomics/templates/base.html
+++ b/web/src/padelnomics/templates/base.html
@@ -29,15 +29,16 @@
   <link rel="alternate" hreflang="de" href="{{ config.BASE_URL }}/de{{ path_suffix }}">
   <link rel="alternate" hreflang="x-default" href="{{ config.BASE_URL }}/en{{ path_suffix }}">
   {% endif %}
+  <meta name="twitter:card" content="summary_large_image">
+
+  <script>window.__GEO = {country: "{{ user_country }}", city: "{{ user_city }}"};</script>
+  {% block head %}
   <meta property="og:title" content="{{ config.APP_NAME }}">
   <meta property="og:description" content="">
   <meta property="og:type" content="website">
   <meta property="og:url" content="{{ config.BASE_URL }}{{ request.path }}">
   <meta property="og:image" content="{{ url_for('static', filename='images/logo.png', _external=True) }}">
-  <meta name="twitter:card" content="summary_large_image">
-
-  <script>window.__GEO = {country: "{{ user_country }}", city: "{{ user_city }}"};</script>
-  {% block head %}{% endblock %}
+  {% endblock %}
 </head>
 <body>
   <nav class="nav-bar" id="main-nav">