Skip to content
This repository was archived by the owner on Sep 6, 2021. It is now read-only.

Commit 1a34a96

Browse files
committed
Check if health data preference is disabled before showing healthdata popup
1 parent 94f4f6e commit 1a34a96

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

src/extensions/default/HealthData/HealthDataNotification.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -45,8 +45,8 @@ define(function (require, exports, module) {
4545
// Check whether the notification dialog should be shown. It will be shown one time. Does not check in testing environment.
4646
if (!params.get("testEnvironment")) {
4747
var alreadyShown = PreferencesManager.getViewState("healthDataNotificationShown");
48-
49-
if (!alreadyShown) {
48+
var prefs = PreferencesManager.getExtensionPrefs("healthData");
49+
if (!alreadyShown && prefs.get("healthDataTracking")) {
5050
HealthDataPopup.showFirstLaunchTooltip()
5151
.done(function () {
5252
PreferencesManager.setViewState("healthDataNotificationShown", true);

0 commit comments

Comments
 (0)