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-2189: Learn More link takes you to Ghostery Image url instead of site #625

Merged
merged 14 commits into from Oct 28, 2020
+21 −21
Merged

GH-2189: Learn More link takes you to Ghostery Image url instead of site #625

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
691e7cf
Update Learn More link in the upgrade banner promo modal
wlycdgr Oct 9, 2020
f685166
Merge branch 'develop' into GH-2189
benstrumeyer Oct 27, 2020
cba2a00
Change learn more link in PromoModals.jsx
benstrumeyer Oct 27, 2020
5ad0534
Change checkout page in Subscribe.jsx and Stats.jsx
benstrumeyer Oct 27, 2020
4c9e9b0
Update historical stats learn more link
benstrumeyer Oct 27, 2020
d404814
Remove / to make query params consistent
benstrumeyer Oct 27, 2020
ed8325c
Merge in develop
wlycdgr Oct 27, 2020
7a8e449
Update link in Subscribe.jsx to use the GHOSTERY_BASE_URL and add bac…
benstrumeyer Oct 28, 2020
ae8c66f
Use GHOSTERY_BASE_URL in StatsView.jsx
benstrumeyer Oct 28, 2020
d98467a
Underline learn more
benstrumeyer Oct 28, 2020
4a68913
Merge branch 'GH-2189' of github.com:ghostery/ghostery-extension into…
benstrumeyer Oct 28, 2020
f3a42ec
Add utm parameters to learn more link in StatsView.jsx
benstrumeyer Oct 28, 2020
389a4b9
Merge in develop
wlycdgr Oct 28, 2020
3f40017
Merge branch 'develop' into GH-2189
christophertino Oct 28, 2020
File filter
Filter file types
Jump to
Jump to file
Failed to load files.

Always

Just for now

Next
Loading status checks…

Update Learn More link in the upgrade banner promo modal

  • Loading branch information
@wlycdgr
wlycdgr committed Oct 9, 2020
commit 691e7cf7b432347fc8b5276f8e573dce3af6e36d
2 app/panel/components/Subscribe.jsx
View file
Open in desktop
@@ -38,7 +38,7 @@ const Subscribe = ({ match }) => {
<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/pricing/">
This conversation was marked as resolved by wlycdgr

This comment has been minimized.

Sign in to view
@wlycdgr

wlycdgr Oct 27, 2020
Member

Let's restore the utm params here

This comment has been minimized.

Sign in to view
@wlycdgr

wlycdgr Oct 27, 2020
Member

Also, let's update the url to use GHOSTERY_BASE_URL to link to stage/prod as appropriate (as per Tino)

This comment has been minimized.

Sign in to view
@benstrumeyer

benstrumeyer Oct 28, 2020
Author Contributor

Restored and updated link!
<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.