Skip to content

feat: say what a program page apply button pays - #151

Merged
sonpiaz merged 1 commit into
mainfrom
fix/program-apply-cta
Oct 2, 2026
Merged

sonpiaz merged 1 commit into
mainfrom
fix/program-apply-cta

Conversation

@sonpiaz

@sonpiaz sonpiaz commented Oct 2, 2026

Copy link
Copy Markdown
Member

Why

Program pages said "Apply to program" and "Join Program". The signed label is "Apply & earn {commission}%": no em dash, at most four words.

Both buttons on /programs/{slug} now use that label. A percentage renders as "Apply & earn 20%". A flat fee renders as "Apply & earn $36" and does not gain a percent sign. An unknown or long rate stays "Apply & earn".

Clicking either button still records outbound_click and also records cta_oa_program.

Test

npx tsx --test scripts/apply-cta.test.ts — 3 passed.

The program pages asked people to apply or join without the commission. The button now says Apply and earn, plus the rate when that stays within four words, and records the click as cta_oa_program.
@vercel

vercel Bot commented Oct 2, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
open-affiliate Ready Ready Preview Oct 2, 2026 8:33pm UTC

Request Review

@sonpiaz sonpiaz added the do-not-merge Emergency stop: never auto-merge this pull request label Oct 2, 2026
@github-actions github-actions Bot added the automerge:blocked Machine policy requires human review label Oct 2, 2026
@github-actions

github-actions Bot commented Oct 2, 2026

Copy link
Copy Markdown
Contributor

Auto-merge policy — LIVE

⛔ Blocked

  • blocking label: do-not-merge
  • changes more than 3 program files
  • sensitive or non-program path: .github/workflows/ci.yml
  • sensitive or non-program path: scripts/apply-cta.test.ts
  • sensitive or non-program path: src/app/programs/[slug]/page.tsx
  • sensitive or non-program path: src/app/programs/[slug]/track-view.tsx
  • sensitive or non-program path: src/lib/apply-cta.ts

Human review is required; auto-merge was not requested.

The workflow never executes pull-request code with its write token
and never falls back to a direct merge.

@sonpiaz
sonpiaz merged commit 9d343ad into main Oct 2, 2026
7 of 8 checks passed

This branch was successfully deployed

1 active deployment
Preview — 3674a578 Deployed Oct 2, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge:blocked Machine policy requires human review do-not-merge Emergency stop: never auto-merge this pull request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant