Skip to content

fix: replace report sharing with marketing CTAs - #62

Merged
blimmer merged 2 commits into
mainfrom
feature/pw-481-remove-waitlist-language-from-the-free-analysis-flow
Sep 21, 2026
Merged

blimmer merged 2 commits into
mainfrom
feature/pw-481-remove-waitlist-language-from-the-free-analysis-flow

Conversation

@blimmer

@blimmer blimmer commented Sep 21, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Removes the report-sharing prompt and uploader so generated reports remain local. The report now links to the PatchWave marketing site with source, medium, and placement-specific UTM tags. PW-481.

Review focus

The main behavior change is intentional: completing an analysis now opens the local report without asking for an email address or offering to upload it.

Before and after

Before

flowchart TD
    A[Run patchwave-analysis] --> B[Scan GitHub organization]
    B --> C[Generate and open local HTML report]
    C --> D{Share report with PatchWave?}
    D -->|No| E[Keep report local]
    D -->|Yes| F[Collect email address]
    F --> G[Upload HTML report]
    G --> H[Add uploader to Loops audience]
    H --> I[Potential waitlist email automation]
    C --> J[Report waitlist CTA]
    J --> K[patchwave.ai]
Loading

After

flowchart TD
    A[Run patchwave-analysis] --> B[Scan GitHub organization]
    B --> C[Generate local HTML report]
    C --> D[Open report in default browser]
    D --> E[Report remains on the user's machine]
    E --> F[PatchWave link with UTM attribution]
    F --> G[patchwave.ai]
Loading

Commits

  • be674dc — replace report sharing with marketing CTAs

@blimmer
blimmer force-pushed the feature/pw-481-remove-waitlist-language-from-the-free-analysis-flow branch from 432b4e3 to d938f01 Compare September 21, 2026 19:36
@blimmer
blimmer force-pushed the feature/pw-481-remove-waitlist-language-from-the-free-analysis-flow branch from d938f01 to be674dc Compare September 21, 2026 19:36
@blimmer blimmer changed the title fix: replace report sharing with signup CTAs fix: replace report sharing with marketing CTAs Sep 21, 2026
@@ -0,0 +1,7 @@
const REPORT_QUERY = 'utm_source=patchwave-analysis&utm_medium=report';

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Adding some UTM params for PostHog

@blimmer
blimmer marked this pull request as ready for review September 21, 2026 19:40
@blimmer
blimmer requested a review from jcarver989 as a code owner September 21, 2026 19:40
@blimmer
blimmer merged commit 3c37b5b into main Sep 21, 2026
9 checks passed
@blimmer
blimmer deleted the feature/pw-481-remove-waitlist-language-from-the-free-analysis-flow branch September 21, 2026 19:46
blimmer pushed a commit that referenced this pull request Sep 21, 2026
🤖 I have created a release *beep* *boop*
---


##
[0.7.1](v0.7.0...v0.7.1)
(2026-09-21)


### Bug Fixes

* replace report sharing with marketing CTAs
([#62](#62))
([3c37b5b](3c37b5b))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: contextbridge-pr-automation[bot] <259134118+contextbridge-pr-automation[bot]@users.noreply.github.com>
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