Skip to content

Commit

Permalink
don't show alert if notification permissions are denied
Browse files Browse the repository at this point in the history
If notification permissions are denied, the description text will be updated telling the user they will not receive reminders or alerts. We do not need to also put this text in an alert (SnackBar)
  • Loading branch information
JGreenlee committed Oct 7, 2024
1 parent 0ffee6a commit 5209b46
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion www/js/usePermissionStatus.ts
Original file line number Diff line number Diff line change
Expand Up @@ -231,7 +231,7 @@ const usePermissionStatus = () => {
return checkOrFix(
appAndChannelNotificationsCheck,
window['cordova'].plugins.BEMDataCollection.fixShowNotifications,
true,
false,
).then((error) => {

Check warning on line 235 in www/js/usePermissionStatus.ts

View check run for this annotation

Codecov / codecov/patch

www/js/usePermissionStatus.ts#L235

Added line #L235 was not covered by tests
if (error) {
appAndChannelNotificationsCheck.desc = error;

Check warning on line 237 in www/js/usePermissionStatus.ts

View check run for this annotation

Codecov / codecov/patch

www/js/usePermissionStatus.ts#L237

Added line #L237 was not covered by tests
Expand Down

0 comments on commit 5209b46

Please sign in to comment.