Skip to content

Toolify engagement: safe embedding, shared-state validation and reliable analytics - #39

Closed
appdevelopsk wants to merge 138 commits into
mainfrom
growth/tool-detail-recent-return-20261105
Closed

appdevelopsk wants to merge 138 commits into
mainfrom
growth/tool-detail-recent-return-20261105

Conversation

@appdevelopsk

@appdevelopsk appdevelopsk commented Nov 5, 2026 •

Copy link
Copy Markdown
Owner

Draft PR #39 — code changes only, not merged or deployed

  • Result-first related/recent tool navigation while retaining existing ad placements.
  • Percentage calculator A/B scenarios, optional local saved draft, multilingual UI.
  • Shared URL protections: default-private text inputs, sensitive input exclusions, malformed and oversized state checks, typed checkbox/radio/select/numeric restoration, stale result-link removal, Next.js history state preservation.
  • Locale-safe search indexes and deduplicated recommendations.
  • GA4: trusted user actions only; reset one-shot telemetry per tool; cancel delayed events on navigation.
  • Latest: embedded iframe-and-attribution HTML now uses an isolated buildToolEmbedSnippet helper with entity escaping for localized titles and URL attributes, including ampersands and quotes. Added 3 Vitest regression cases in site/src/lib/tools/embed-html.test.ts.

Evidence and blockers

  • GitHub readback confirmed ShareBar calls the shared HTML helper and test source is committed.
  • Branch 138 commits ahead of main, 0 behind, 34 changed files; no successful statuses or reviews reported.
  • Direct git ls-remote fails DNS in the available container, so full npm ci, repository TypeScript check, Vitest, production Next.js build, device/browser testing and GA4 DebugView remain unverified. GitHub Actions deliberately unused.
  • Verification log: docs/PR39_VERIFICATION_2026-11-05.md.

NO-GO / keep draft

In an owner-accessible clone, run bash scripts/check-engagement-release.sh, review generated HTML on an external host page, and verify share restoration, mobile/desktop, 17 locales, ads, privacy/consent, GA4 events and client navigation. Obtain code review before merge. Owner performs deployment; no confirmed production engagement or revenue improvement.

@appdevelopsk appdevelopsk changed the title Improve return-visitor tool navigation on detail pages Improve return-visitor tool navigation and measure recent-tool clicks Nov 5, 2026
@appdevelopsk appdevelopsk changed the title Improve return-visitor tool navigation and measure recent-tool clicks Engagement: recent tools + percentage A/B comparison and opt-in resume Nov 5, 2026
@appdevelopsk appdevelopsk changed the title Engagement: recent tools + percentage A/B comparison and opt-in resume Toolify engagement: recent navigation, percentage A/B, opt-in resume, privacy-safe tracking Nov 5, 2026
@appdevelopsk appdevelopsk changed the title Toolify engagement: reliable GA4 measurement, safe sharing and localized navigation Toolify engagement: safe sharing, reliable GA4 and localized tool navigation Nov 6, 2026
@appdevelopsk appdevelopsk changed the title Toolify engagement: safe sharing, reliable GA4 and localized tool navigation Toolify engagement: safe shared results, reliable telemetry, localized navigation Nov 6, 2026
@appdevelopsk appdevelopsk changed the title Toolify engagement: safe shared results, reliable telemetry, localized navigation Toolify engagement: private-by-default shared text, reliable analytics and localized navigation Nov 6, 2026
@appdevelopsk appdevelopsk changed the title Toolify engagement: private-by-default shared text, reliable analytics and localized navigation Toolify engagement: validated shared-state restore, privacy and reliable analytics Nov 6, 2026
@appdevelopsk appdevelopsk changed the title Toolify engagement: validated shared-state restore, privacy and reliable analytics Toolify engagement: safe embedding, shared-state validation and reliable analytics Nov 6, 2026
@appdevelopsk

Copy link
Copy Markdown
Owner Author

PR #40 に統合してマージ済みのため閉じます。

appdevelopsk added a commit that referenced this pull request Nov 6, 2026
…ngagement-20261106

Integration draft: Toolify growth PR #38 + engagement PR #39
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.

1 participant