Skip to content

/ ghostery-extension

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

GH-2072: Add subscription interval to ping params and change UpgradePlan CTA link params #586

Merged
merged 8 commits into from Jul 27, 2020
+28 −14
Merged

GH-2072: Add subscription interval to ping params and change UpgradePlan CTA link params #586

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
0d645c8
Delete unnecesssary ping params
benstrumeyer Jul 24, 2020
6d1d37b
Add subscription interval to ping params
benstrumeyer Jul 27, 2020
247554f
Change subscription_interval to interval to match the query param in …
benstrumeyer Jul 27, 2020
e873160
Remove null check
benstrumeyer Jul 27, 2020
23fb87b
Change variable name
benstrumeyer Jul 27, 2020
6f72d1b
Fix linter
benstrumeyer Jul 27, 2020
f554c7d
Update comment. Wrap ternary in (), update subscripton_interval ping …
benstrumeyer Jul 27, 2020
2574305
Add semicolon
benstrumeyer Jul 27, 2020
File filter
Filter file types
Jump to
Jump to file
Failed to load files.

Always

Just for now

Next

Delete unnecesssary ping params

  • Loading branch information
@benstrumeyer
benstrumeyer committed Jul 24, 2020
commit 0d645c87ffb98ab8b75959f4d1e0d6edf17e20f6
12 app/hub/Views/UpgradePlanView/UpgradePlanView.jsx
View file
Open in desktop
@@ -166,16 +166,10 @@ const UpgradePlanView = (props) => {
mobileComparisonTableRef.current.scrollIntoView({ behavior: 'smooth' });
};

// UTM params
const signedIn = +!!user;
const subscriptionType = () => {
if (isPremium) return 'PREMIUM';
if (isPlus) return 'SUPPORTER';
return '-1';
};
// Interval is the query Param to show monthly/yearly pricing in checkout web, also used as a ping parameter
// UTM and Query Params
// Subscription_interval is the query param to show monthly/yearly pricing in checkout web
This conversation was marked as resolved by wlycdgr

This comment has been minimized.

Sign in to view
@wlycdgr

wlycdgr Jul 27, 2020
Member

Let's update the comment to use the updated interval param name

This comment has been minimized.

Sign in to view
@benstrumeyer

benstrumeyer Jul 27, 2020
Author Contributor

Nice catch!
const interval = show_yearly_prices ? 'year' : 'month';
const utmParams = `utm_source=gbe&signedIn=${signedIn}&st=${subscriptionType()}&subscription_interval=${interval}`;
const utmParams = `utm_source=gbe&subscription_interval=${interval}`;

const plusCTAButton = (position) => {
const utm_campaign = position === 'top' ? 'c_1' : 'c_2';
This conversation was marked as resolved by wlycdgr

This comment has been minimized.

Sign in to view
@wlycdgr

wlycdgr Jul 27, 2020
Member

Could you throw a set of parens around the rvalue to make this a little easier to parse visually?

This comment has been minimized.

Sign in to view
@benstrumeyer

benstrumeyer Jul 27, 2020
Author Contributor

Yes, yes I can :D
ProTip! Use n and p to navigate between commits in a pull request.