From 98abe5a0de388267f0fb5815c48024e8ab7d5e9a Mon Sep 17 00:00:00 2001 From: giles Date: Tue, 10 Feb 2026 20:20:49 +0000 Subject: [PATCH] fix: rename post slug URL param to page_slug to avoid collision The market blueprint already uses for product routes, so the app-level prefix uses instead. Co-Authored-By: Claude Opus 4.6 --- app.py | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/app.py b/app.py index e1d52e4..90d1d91 100644 --- a/app.py +++ b/app.py @@ -54,27 +54,27 @@ def create_app() -> "Quart": app.jinja_loader, ]) - # Market blueprint nested under post slug: /// + # Market blueprint nested under post slug: /// app.register_blueprint( register_market_bp( url_prefix="/", title=config()["coop_title"], ), - url_prefix="//", + url_prefix="//", ) - # --- Auto-inject slug and market_slug into url_for() calls --- + # --- Auto-inject page_slug and market_slug into url_for() calls --- @app.url_value_preprocessor def pull_slugs(endpoint, values): if values: - if "slug" in values: - g.post_slug = values.pop("slug") + if "page_slug" in values: + g.post_slug = values.pop("page_slug") if "market_slug" in values: g.market_slug = values.pop("market_slug") @app.url_defaults def inject_slugs(endpoint, values): - for attr, param in [("post_slug", "slug"), ("market_slug", "market_slug")]: + for attr, param in [("post_slug", "page_slug"), ("market_slug", "market_slug")]: val = g.get(attr) if val and param not in values: if app.url_map.is_endpoint_expecting(endpoint, param):