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

Commit fa4e6c5

Browse files
committed
Merge pull request #11950 from adobe/abose/healthNoPopup
Check if health data preference is disabled before showing healthdata… Looks good to me @abose . Merging now
2 parents 94f4f6e + 1a34a96 commit fa4e6c5

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)