Repository navigation
feat(growth): route editorial-guide readers into matching app catalog filters - #71
Merged
Merged
Conversation
appdevelopsk
marked this pull request as ready for review
November 6, 2026 15:23
7 tasks
appdevelopsk
added a commit
that referenced
this pull request
Nov 7, 2026
… and unit gates The deployed pages were correct but the gates still pinned old values: - check_growth_live: #77 input handler (clears dedupe key, never reports per keystroke) - check_app_discovery_live: #78 Debate Me title, #53 robots Allow: /icons/ - catalog-growth: #71 ?goal= landing needs URL/location/history/reset in sandbox; #63 app_detail_click - debate-me / dotchi: current title; sitemap lastmod must not regress (not equal a fixed date) python 72/72, node tests 52/52 (was 47/52 on main), live 92/92. Claude-Session: https://claude.ai/code/session_01J5anYNmtTi1EKefKjB5aoV
appdevelopsk
added a commit
that referenced
this pull request
Nov 7, 2026
…78/#79 (#83) * fix(deploy): publish TaleTime and Dog Breeds support pages updated in #78/#79 sitemap.xml (#81) advertises lastmod 2026-11-07 for both pages, but publish_app_discovery.sh did not list them, so --apply would ship the new sitemap while leaving their old HTML live. Claude-Session: https://claude.ai/code/session_01J5anYNmtTi1EKefKjB5aoV * test(release): follow intentional #53/#63/#71/#77/#78 changes in live and unit gates The deployed pages were correct but the gates still pinned old values: - check_growth_live: #77 input handler (clears dedupe key, never reports per keystroke) - check_app_discovery_live: #78 Debate Me title, #53 robots Allow: /icons/ - catalog-growth: #71 ?goal= landing needs URL/location/history/reset in sandbox; #63 app_detail_click - debate-me / dotchi: current title; sitemap lastmod must not regress (not equal a fixed date) python 72/72, node tests 52/52 (was 47/52 on main), live 92/92. Claude-Session: https://claude.ai/code/session_01J5anYNmtTi1EKefKjB5aoV
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.
Four editorial guides currently link readers to an unfiltered 92-app catalog, forcing them to reselect the category they were reading about. Add allowlisted
?goal=category deep links for quick games, daily habits, decision helpers and AI tools. Preserve the plain /apps breadcrumb; initialize the catalog's chosen filter button and results before first render. No raw search terms or PII in the URL. Addscripts/test_guide_catalog_links.py(4 offline source test methods) to detect missing guide links, missing catalog allowlist, lost filter controls and missing 92 cards/5 spotlight links. GitHub branch source reviewed with all four category pairs, each one filtered and one unfiltered link, exact 92 cards, five spotlight links, and four matching filter options. Python tests were not executed in this environment because Github connector content could not be transferred in full to the isolated runtime and raw.github.com DNS was unavailable. No GitHub Actions, deployment or store updates.