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-1947 Plus checkout UTM params #499

Merged
merged 4 commits into from Feb 21, 2020
+28 −17
Merged

GH-1947 Plus checkout UTM params #499

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
da2976a
Add UTM params to Plus checkout from all locations
benstrumeyer Feb 18, 2020
190c99a
Update tests
benstrumeyer Feb 19, 2020
e06e516
Remove / from links with utm params
benstrumeyer Feb 19, 2020
1f5c6c8
Refactor messaging passing
benstrumeyer Feb 21, 2020
File filter
Filter file types
Clear filters
Jump to
Jump to file
Failed to load files.

Always

Just for now

Prev Next
Loading status checks…
Remove / from links with utm params
  • Loading branch information
@benstrumeyer
benstrumeyer committed Feb 19, 2020
commit e06e5167e6f0a9cbb3bc46202161236cfcb5612b
2 _locales/en/messages.json
View file
Open in desktop
@@ -1936,7 +1936,7 @@
"content": "</a>"
},
"link_start": {
"content": "<br><a target=\"_blank\" rel=\"noopener noreferrer\" href=\"https://www.ghostery.com/products/plus/?utm_source=gbe&utm_campaign=in_app_hstats_lm\">"
"content": "<br><a target=\"_blank\" rel=\"noopener noreferrer\" href=\"https://www.ghostery.com/products/plus?utm_source=gbe&utm_campaign=in_app_hstats_lm\">"
}
}
},
2 app/hub/Views/PlusView/PlusView.jsx
View file
Open in desktop
@@ -29,7 +29,7 @@ class PlusView extends Component {
*/
_renderButton = (additionalClasses) => {
const { isPlus, onPlusClick } = this.props;
const buttonHref = `https://checkout.${globals.GHOSTERY_DOMAIN}.com/plus/?utm_source=gbe&utm_campaign=intro_hub_plus`;
const buttonHref = `https://checkout.${globals.GHOSTERY_DOMAIN}.com/plus?utm_source=gbe&utm_campaign=intro_hub_plus`;
const buttonClassNames = ClassNames('PlusView__button', 'button', additionalClasses, {
disabled: isPlus,
});
2 app/hub/Views/PlusView/__tests__/PlusView.test.jsx
View file
Open in desktop
@@ -70,7 +70,7 @@ describe('app/hub/Views/PlusView component', () => {
component.find('.PlusView__button').first().simulate('click');
expect(initialState.onPlusClick.mock.calls.length).toBe(1);

expect(component.find('.PlusView__button').first().props().href).toBe('https://checkout.ghosterystage.com/plus/?utm_source=gbe&utm_campaign=intro_hub_plus')
expect(component.find('.PlusView__button').first().props().href).toBe('https://checkout.ghosterystage.com/plus?utm_source=gbe&utm_campaign=intro_hub_plus')

expect(component.find('.PlusView__button').length).toBe(8);
expect(component.find('.PlusView__button.disabled').length).toBe(0);
16 app/hub/Views/PlusView/__tests__/__snapshots__/PlusView.test.jsx.snap
View file
Open in desktop
@@ -361,7 +361,7 @@ exports[`app/hub/Views/PlusView component Snapshot tests with react-test-rendere
>
<a
className="PlusView__button button"
href="https://checkout.ghosterystage.com/plus/?utm_source=gbe&utm_campaign=intro_hub_plus"
href="https://checkout.ghosterystage.com/plus?utm_source=gbe&utm_campaign=intro_hub_plus"
onClick={[Function]}
rel="noopener noreferrer"
target="_blank"
@@ -468,7 +468,7 @@ exports[`app/hub/Views/PlusView component Snapshot tests with react-test-rendere
</div>
<a
className="PlusView__button button show-for-large"
href="https://checkout.ghosterystage.com/plus/?utm_source=gbe&utm_campaign=intro_hub_plus"
href="https://checkout.ghosterystage.com/plus?utm_source=gbe&utm_campaign=intro_hub_plus"
onClick={[Function]}
rel="noopener noreferrer"
target="_blank"
@@ -497,7 +497,7 @@ exports[`app/hub/Views/PlusView component Snapshot tests with react-test-rendere
/>
<a
className="PlusView__button button PlusView--addSideMargin"
href="https://checkout.ghosterystage.com/plus/?utm_source=gbe&utm_campaign=intro_hub_plus"
href="https://checkout.ghosterystage.com/plus?utm_source=gbe&utm_campaign=intro_hub_plus"
onClick={[Function]}
rel="noopener noreferrer"
target="_blank"
@@ -540,7 +540,7 @@ exports[`app/hub/Views/PlusView component Snapshot tests with react-test-rendere
</div>
<a
className="PlusView__button button show-for-large"
href="https://checkout.ghosterystage.com/plus/?utm_source=gbe&utm_campaign=intro_hub_plus"
href="https://checkout.ghosterystage.com/plus?utm_source=gbe&utm_campaign=intro_hub_plus"
onClick={[Function]}
rel="noopener noreferrer"
target="_blank"
@@ -569,7 +569,7 @@ exports[`app/hub/Views/PlusView component Snapshot tests with react-test-rendere
/>
<a
className="PlusView__button button PlusView--addSideMargin"
href="https://checkout.ghosterystage.com/plus/?utm_source=gbe&utm_campaign=intro_hub_plus"
href="https://checkout.ghosterystage.com/plus?utm_source=gbe&utm_campaign=intro_hub_plus"
onClick={[Function]}
rel="noopener noreferrer"
target="_blank"
@@ -603,7 +603,7 @@ exports[`app/hub/Views/PlusView component Snapshot tests with react-test-rendere
</div>
<a
className="PlusView__button button show-for-large"
href="https://checkout.ghosterystage.com/plus/?utm_source=gbe&utm_campaign=intro_hub_plus"
href="https://checkout.ghosterystage.com/plus?utm_source=gbe&utm_campaign=intro_hub_plus"
onClick={[Function]}
rel="noopener noreferrer"
target="_blank"
@@ -632,7 +632,7 @@ exports[`app/hub/Views/PlusView component Snapshot tests with react-test-rendere
/>
<a
className="PlusView__button button PlusView--addSideMargin"
href="https://checkout.ghosterystage.com/plus/?utm_source=gbe&utm_campaign=intro_hub_plus"
href="https://checkout.ghosterystage.com/plus?utm_source=gbe&utm_campaign=intro_hub_plus"
onClick={[Function]}
rel="noopener noreferrer"
target="_blank"
@@ -682,7 +682,7 @@ exports[`app/hub/Views/PlusView component Snapshot tests with react-test-rendere
/>
<a
className="PlusView__button button PlusView--addSideMargin"
href="https://checkout.ghosterystage.com/plus/?utm_source=gbe&utm_campaign=intro_hub_plus"
href="https://checkout.ghosterystage.com/plus?utm_source=gbe&utm_campaign=intro_hub_plus"
onClick={[Function]}
rel="noopener noreferrer"
target="_blank"
2 app/panel/components/Subscribe.jsx
View file
Open in desktop
@@ -36,7 +36,7 @@ const Subscribe = (props) => {
<div className="pitch-container">
<span className="pitch-text" dangerouslySetInnerHTML={{ __html: t('subscribe_pitch') }} />
</div>
<PanelToTabLink href="https://www.ghostery.com/products/plus/?utm_source=gbe&utm_campaign=in_app_plus_lm">
<PanelToTabLink href="https://www.ghostery.com/products/plus?utm_source=gbe&utm_campaign=in_app_plus_lm">
<span className="pitch-learn-more">{t('subscribe_pitch_learn_more')}</span>
</PanelToTabLink>
<div>
ProTip! Use n and p to navigate between commits in a pull request.