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-2071 A/B testing console error #588

Merged
merged 1 commit into from Jul 28, 2020
+1 −1
Merged

GH-2071 A/B testing console error #588

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter
Filter file types
Jump to
Jump to file
Failed to load files.

Always

Just for now

Loading status checks…

Fix bug where pm is undefined

  • Loading branch information
@benstrumeyer
benstrumeyer committed Jul 28, 2020
commit 76e597478bc777a9814614c6bc205dd2b9baffb1
2 app/hub/Views/HomeView/HomeViewContainer.jsx
View file
Open in desktop
@@ -119,7 +119,7 @@ class HomeViewContainer extends Component {
} = home;

// Flag to display promo modal (used in A/B testing)
const { pm } = QueryString.parse(window.location.search);
const pm = QueryString.parse(window.location.search).pm || false;
// Logic to display premium modal if it is the case that it is being shown once per hub refresh to non-premium users
const showPromoModal = pm && pm === 'true' && !premium_promo_modal_shown && !isPremium;

ProTip! Use n and p to navigate between commits in a pull request.