Automattic / Automattic/jetpack

Align Jetpack premium block upgrade nudge event with other Calypso upgrade nudges

Open
#14,392 3 comments 0 reactions 0 assignees View on GitHub
[Block] Pay With Paypal [Block] Payment Button [Feature] Pay with PayPal [Focus] Blocks Task
Dominant language
PHP
Stars
1.8k
Forks
898
Avg merge
1d 18h
Merged PRs (30d)
774

Description

Having different names and props makes it hard to compare how CTAs perform against each other.

We should have the same event name with other CTAs in Calypso and we'll segment by property.

In Jetpack currently:
https://github.com/Automattic/jetpack/blob/03212a4ae7fe3fe060c231d00702d1b7eecf8dd1/extensions/shared/components/upgrade-nudge/index.jsx#L95-L98

In Calypso they look more like:
```js
recordTracks( 'calypso_upgrade_nudge_cta_click', {
cta_name: event,
cta_feature: feature,
cta_size: 'regular',
} );
```

See p9Jlb4-17c-p2/#comment-1798

Contributor guide

Open the contributing guide

Research direction

Start in extensions/shared/components/upgrade-nudge/index.jsx at the referenced event-tracking code, then compare it with the Calypso upgrade-nudge example in the issue. Done means the Jetpack premium block upgrade nudge uses the shared event name and the cta_name, cta_feature, and cta_size properties described there.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript, react
Domain
analytics, frontend
Issue type
Refactor
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.