Skip to content
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

GH-2040: Update Insights in-app promo CTA link #594

Closed
wants to merge 4 commits into from
Closed

Conversation

@caseyoppenheim
Copy link

@caseyoppenheim caseyoppenheim commented Aug 11, 2020

  • Have you followed the guidelines in CONTRIBUTING.md?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • Have you added an explanation of what your changes do?
  • Does your submission pass tests?
  • Did you lint your code prior to submission?
@caseyoppenheim caseyoppenheim requested review from christophertino, jsignanini, wlycdgr, zarembsky and ghostery/ghostery as code owners Aug 11, 2020
@fcjr fcjr changed the base branch from master to develop Aug 11, 2020
@fcjr fcjr changed the title GD2040: Update Insights in-app promo CTA link GD-2040: Update Insights in-app promo CTA link Aug 11, 2020
app/shared-components/PromoModal/PromoModal.jsx Outdated Show resolved Hide resolved
app/shared-components/PromoModal/PromoModal.jsx Outdated Show resolved Hide resolved
@fcjr fcjr changed the title GD-2040: Update Insights in-app promo CTA link GH-2040: Update Insights in-app promo CTA link Aug 12, 2020
@fcjr
fcjr approved these changes Aug 12, 2020
@@ -76,7 +76,7 @@ class PromoModal extends React.Component {
break;
case INSIGHTS:
sendMessage('ping', 'promo_modals_insights_upgrade_cta');
url = `https://checkout.${DOMAIN}.com/insights?utm_source=gbe&utm_campaign=${utm_campaign}`;
url = `https://www.ghostery.com/insights/?utm_source=gbe&utm_campaign=${utm_campaign}`;

This comment has been minimized.

@fcjr

fcjr Aug 13, 2020
Member

looks like we do actually have a staging website now, rather than doubling dev work lets wait till support for this is done by @benstrumeyer in #595 to update this PR.

@fcjr
Copy link
Member

@fcjr fcjr commented Aug 13, 2020

Closing in favor of #595

@fcjr fcjr closed this Aug 13, 2020
@fcjr fcjr deleted the GD2040 branch Aug 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants