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
Clear filters
Jump to
Jump to file
Failed to load files.

Always

Just for now

Prev
Loading status checks…

Add semicolon

  • Loading branch information
@benstrumeyer
benstrumeyer committed Jul 27, 2020
commit 2574305ac6e14cb2274e5617b7d8c61857136aff
2 src/background.js
View file
Open in desktop
@@ -823,7 +823,7 @@ function onMessageHandler(request, sender, callback) {
.catch((err) => {
log('LOGIN ERROR', err);
callback({ errors: _getJSONAPIErrorsObject(err) });
})
});
return true;
}
if (name === 'account.register') {
ProTip! Use n and p to navigate between commits in a pull request.