MOBILE-4313 notifications: Show warnings if permissions missing

dpalou · dpalou · commit edd6214e71f5 · 2024-02-19T15:58:56.000+01:00
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -81,7 +81,7 @@
     "@moodlehq/cordova-plugin-inappbrowser": "5.0.0-moodle.3",
     "@moodlehq/cordova-plugin-intent": "2.2.0-moodle.3",
     "@moodlehq/cordova-plugin-ionic-webview": "5.0.0-moodle.3",
-    "@moodlehq/cordova-plugin-local-notification": "0.9.0-moodle.11",
+    "@moodlehq/cordova-plugin-local-notification": "0.9.0-moodle.12",
     "@moodlehq/cordova-plugin-qrscanner": "3.0.1-moodle.5",
     "@moodlehq/cordova-plugin-statusbar": "4.0.0-moodle.3",
     "@moodlehq/cordova-plugin-zip": "3.1.0-moodle.1",
diff --git a/scripts/langindex.json b/scripts/langindex.json
@@ -1752,6 +1752,8 @@
   "core.errorsyncblocked": "local_moodlemobileapp",
   "core.errorurlschemeinvalidscheme": "local_moodlemobileapp",
   "core.errorurlschemeinvalidsite": "local_moodlemobileapp",
+  "core.exactalarmsturnedoff": "local_moodlemobileapp",
+  "core.exactalarmsturnedoffmessage": "local_moodlemobileapp",
   "core.expand": "moodle",
   "core.explanationdigitalminor": "moodle",
   "core.favourites": "moodle",
@@ -2225,6 +2227,7 @@
   "core.notenrolledprofile": "moodle",
   "core.notice": "moodle",
   "core.notingroup": "moodle",
+  "core.notnow": "local_moodlemobileapp",
   "core.notsent": "local_moodlemobileapp",
   "core.now": "moodle",
   "core.nummore": "local_moodlemobileapp",
@@ -2499,6 +2502,10 @@
   "core.today": "moodle",
   "core.toggledelete": "local_moodlemobileapp",
   "core.tryagain": "local_moodlemobileapp",
+  "core.turnon": "local_moodlemobileapp",
+  "core.turnonexactalarms": "local_moodlemobileapp",
+  "core.turnonnotifications": "local_moodlemobileapp",
+  "core.turnonnotificationsmessage": "local_moodlemobileapp",
   "core.twoparagraphs": "local_moodlemobileapp",
   "core.type": "repository",
   "core.uhoh": "local_moodlemobileapp",
diff --git a/src/addons/calendar/pages/event/event.html b/src/addons/calendar/pages/event/event.html
@@ -120,6 +120,23 @@ <h1>
             </ion-item>
         </ion-list>
 
+        <ion-card class="core-warning-card core-card-with-buttons"
+            *ngIf="remindersEnabled && event && !canScheduleExactAlarms && !scheduleExactWarningHidden">
+            <ion-item class="ion-text-wrap">
+                <ion-icon name="fas-circle-info" slot="start" aria-hidden="true" />
+                <ion-label>
+                    <p><strong>{{ 'core.exactalarmsturnedoff' | translate }}</strong></p>
+                    <p>{{ 'core.exactalarmsturnedoffmessage' | translate }}</p>
+                </ion-label>
+            </ion-item>
+            <div class="core-card-buttons">
+                <ion-button fill="clear" (click)="hideAlarmWarning()">
+                    {{ 'core.dontshowagain' | translate | coreNoPeriod }}
+                </ion-button>
+                <ion-button fill="outline" (click)="openAlarmSettings()">{{ 'core.turnon' | translate }}</ion-button>
+            </div>
+        </ion-card>
+
         <ion-card *ngIf="remindersEnabled && event">
             <ion-item>
                 <ion-label>
diff --git a/src/addons/calendar/pages/event/event.ts b/src/addons/calendar/pages/event/event.ts
@@ -40,6 +40,9 @@ import { AddonCalendarEventsSource } from '@addons/calendar/classes/events-sourc
 import { CoreSwipeNavigationItemsManager } from '@classes/items-management/swipe-navigation-items-manager';
 import { CoreReminders, CoreRemindersService } from '@features/reminders/services/reminders';
 import { CoreRemindersSetReminderMenuComponent } from '@features/reminders/components/set-reminder-menu/set-reminder-menu';
+import { CoreLocalNotifications } from '@services/local-notifications';
+import { CorePlatform } from '@services/platform';
+import { CoreConfig } from '@services/config';
 
 /**
  * Page that displays a single calendar event.
@@ -61,6 +64,7 @@ export class AddonCalendarEventPage implements OnInit, OnDestroy {
     protected defaultTimeChangedObserver: CoreEventObserver;
     protected currentSiteId: string;
     protected updateCurrentTime?: number;
+    protected appResumeSubscription: Subscription;
 
     eventLoaded = false;
     event?: AddonCalendarEventToDisplay;
@@ -78,6 +82,8 @@ export class AddonCalendarEventPage implements OnInit, OnDestroy {
     hasOffline = false;
     isOnline = false;
     syncIcon = CoreConstants.ICON_LOADING; // Sync icon.
+    canScheduleExactAlarms = true;
+    scheduleExactWarningHidden = false;
 
     constructor(
         protected route: ActivatedRoute,
@@ -138,6 +144,11 @@ export class AddonCalendarEventPage implements OnInit, OnDestroy {
         this.updateCurrentTime = window.setInterval(() => {
             this.currentTime = CoreTimeUtils.timestamp();
         }, 5000);
+
+        this.checkExactAlarms();
+        this.appResumeSubscription = CorePlatform.resume.subscribe(() => {
+            this.checkExactAlarms();
+        });
     }
 
     /**
@@ -153,6 +164,14 @@ export class AddonCalendarEventPage implements OnInit, OnDestroy {
         this.reminders = await AddonCalendarHelper.getEventReminders(this.eventId, this.event.timestart, this.currentSiteId);
     }
 
+    /**
+     * Check if the app can schedule exact alarms.
+     */
+    protected async checkExactAlarms(): Promise<void> {
+        this.scheduleExactWarningHidden = !!(await CoreConfig.get(CoreConstants.DONT_SHOW_EXACT_ALARMS_WARNING, 0));
+        this.canScheduleExactAlarms = await CoreLocalNotifications.canScheduleExactAlarms();
+    }
+
     /**
      * @inheritdoc
      */
@@ -616,6 +635,21 @@ export class AddonCalendarEventPage implements OnInit, OnDestroy {
         }
     }
 
+    /**
+     * Open alarm settings.
+     */
+    openAlarmSettings(): void {
+        CoreLocalNotifications.openAlarmSettings();
+    }
+
+    /**
+     * Hide alarm warning.
+     */
+    hideAlarmWarning(): void {
+        CoreConfig.set(CoreConstants.DONT_SHOW_EXACT_ALARMS_WARNING, 1);
+        this.scheduleExactWarningHidden = true;
+    }
+
     /**
      * @inheritdoc
      */
@@ -626,6 +660,7 @@ export class AddonCalendarEventPage implements OnInit, OnDestroy {
         this.onlineObserver.unsubscribe();
         this.newEventObserver.off();
         this.events?.destroy();
+        this.appResumeSubscription.unsubscribe();
         clearInterval(this.updateCurrentTime);
     }
 
diff --git a/src/addons/notifications/pages/list/list.html b/src/addons/notifications/pages/list/list.html
@@ -17,8 +17,23 @@ <h1>{{ 'addon.notifications.notifications' | translate }}</h1>
             <ion-refresher-content pullingText="{{ 'core.pulltorefresh' | translate }}" />
         </ion-refresher>
         <core-loading [hideUntil]="notifications.loaded">
+            <ion-card class="core-warning-card core-card-with-buttons" *ngIf="!hasNotificationsPermission && !permissionWarningHidden">
+                <ion-item class="ion-text-wrap">
+                    <ion-icon name="fas-circle-info" slot="start" aria-hidden="true" />
+                    <ion-label>
+                        <p><strong>{{ 'core.turnonnotifications' | translate }}</strong></p>
+                        <p>{{ 'core.turnonnotificationsmessage' | translate }}</p>
+                    </ion-label>
+                </ion-item>
+                <div class="core-card-buttons">
+                    <ion-button fill="clear" (click)="hidePermissionWarning()">
+                        {{ 'core.dontshowagain' | translate | coreNoPeriod }}
+                    </ion-button>
+                    <ion-button fill="outline" (click)="openSettings()">{{ 'core.turnon' | translate }}</ion-button>
+                </div>
+            </ion-card>
 
-            <ion-item *ngFor="let notification of notifications.items" class="ion-text-wrap"
+            <ion-item *ngFor="let notification of notifications.items" class="ion-text-wrap addon-notification-item"
                 [attr.aria-current]="notifications.getItemAriaCurrent(notification)" (click)="notifications.select(notification)" button
                 [detail]="false" lines="full">
 
diff --git a/src/addons/notifications/pages/list/list.scss b/src/addons/notifications/pages/list/list.scss
@@ -1,6 +1,6 @@
 @use "theme/globals" as *;
 
-ion-item {
+ion-item.addon-notification-item {
     ion-label {
         margin-top: 8px;
         margin-bottom: 8px;
diff --git a/src/addons/notifications/pages/list/list.ts b/src/addons/notifications/pages/list/list.ts
@@ -31,6 +31,10 @@ import { CoreTimeUtils } from '@services/utils/time';
 import { AddonNotificationsNotificationsSource } from '@addons/notifications/classes/notifications-source';
 import { CoreListItemsManager } from '@classes/items-management/list-items-manager';
 import { AddonLegacyNotificationsNotificationsSource } from '@addons/notifications/classes/legacy-notifications-source';
+import { CoreLocalNotifications } from '@services/local-notifications';
+import { CoreConfig } from '@services/config';
+import { CoreConstants } from '@/core/constants';
+import { CorePlatform } from '@services/platform';
 
 /**
  * Page that displays the list of notifications.
@@ -47,12 +51,15 @@ export class AddonNotificationsListPage implements AfterViewInit, OnDestroy {
     fetchMoreNotificationsFailed = false;
     canMarkAllNotificationsAsRead = false;
     loadingMarkAllNotificationsAsRead = false;
+    hasNotificationsPermission = true;
+    permissionWarningHidden = false;
 
     protected isCurrentView?: boolean;
     protected cronObserver?: CoreEventObserver;
     protected readObserver?: CoreEventObserver;
     protected pushObserver?: Subscription;
     protected pendingRefresh = false;
+    protected appResumeSubscription?: Subscription;
 
     constructor() {
         try {
@@ -67,7 +74,14 @@ export class AddonNotificationsListPage implements AfterViewInit, OnDestroy {
         } catch(error) {
             CoreDomUtils.showErrorModal(error);
             CoreNavigator.back();
+
+            return;
         }
+
+        this.checkPermission();
+        this.appResumeSubscription = CorePlatform.resume.subscribe(() => {
+            this.checkPermission();
+        });
     }
 
     /**
@@ -120,6 +134,14 @@ export class AddonNotificationsListPage implements AfterViewInit, OnDestroy {
         deepLinkManager.treatLink();
     }
 
+    /**
+     * Check if the app has permission to display notifications.
+     */
+    protected async checkPermission(): Promise<void> {
+        this.permissionWarningHidden = !!(await CoreConfig.get(CoreConstants.DONT_SHOW_NOTIFICATIONS_PERMISSION_WARNING, 0));
+        this.hasNotificationsPermission = await CoreLocalNotifications.hasNotificationsPermission();
+    }
+
     /**
      * Convenience function to get notifications. Gets unread notifications first.
      *
@@ -211,6 +233,21 @@ export class AddonNotificationsListPage implements AfterViewInit, OnDestroy {
         refresher?.complete();
     }
 
+    /**
+     * Open notification settings.
+     */
+    openSettings(): void {
+        CoreLocalNotifications.openNotificationSettings();
+    }
+
+    /**
+     * Hide permission warning.
+     */
+    hidePermissionWarning(): void {
+        CoreConfig.set(CoreConstants.DONT_SHOW_NOTIFICATIONS_PERMISSION_WARNING, 1);
+        this.permissionWarningHidden = true;
+    }
+
     /**
      * User entered the page.
      */
@@ -241,6 +278,7 @@ export class AddonNotificationsListPage implements AfterViewInit, OnDestroy {
         this.readObserver?.off();
         this.pushObserver?.unsubscribe();
         this.notifications?.destroy();
+        this.appResumeSubscription?.unsubscribe();
     }
 
 }
diff --git a/src/core/constants.ts b/src/core/constants.ts
@@ -155,6 +155,9 @@ export class CoreConstants {
 
     // Other constants.
     static readonly CALENDAR_DEFAULT_STARTING_WEEKDAY = 1;
+    static readonly DONT_SHOW_NOTIFICATIONS_PERMISSION_WARNING = 'CoreDontShowNotificationsPermissionWarning';
+    static readonly DONT_SHOW_EXACT_ALARMS_WARNING = 'CoreDontShowScheduleExactWarning';
+    static readonly EXACT_ALARMS_WARNING_DISPLAYED = 'CoreScheduleExactWarningModalDisplayed';
 
     // Config & environment constants.
     static readonly CONFIG = { ...envJson.config } as unknown as EnvironmentConfig; // Data parsed from config.json files.
diff --git a/src/core/lang.json b/src/core/lang.json
@@ -124,6 +124,8 @@
     "errorsyncblocked": "This {{$a}} cannot be synchronised right now because of an ongoing process. Please try again later. If the problem persists, try restarting the app.",
     "errorurlschemeinvalidscheme": "This URL is meant to be used in another app: {{$a}}.",
     "errorurlschemeinvalidsite": "This site URL cannot be opened in this app.",
+    "exactalarmsturnedoff": "Real-time notifications are turned off",
+    "exactalarmsturnedoffmessage": "To make sure you don't miss any important alerts, turn on 'Alarms and reminders' in your device's settings.",
     "expand": "Expand",
     "explanationdigitalminor": "This information is required to determine if your age is over the digital age of consent. This is the age when an individual can consent to terms and conditions and their data being legally stored and processed.",
     "favourites": "Starred",
@@ -225,6 +227,7 @@
     "notenrolledprofile": "This profile is not available because this user is not enrolled in this course.",
     "notice": "Notice",
     "notingroup": "Sorry, but you need to be part of a group to see this page.",
+    "notnow": "Not now",
     "notsent": "Not sent",
     "now": "now",
     "nummore": "{{$a}} more",
@@ -333,6 +336,10 @@
     "today": "Today",
     "toggledelete": "Toggle delete buttons",
     "tryagain": "Try again",
+    "turnon": "Turn on",
+    "turnonexactalarms": "Turn on real-time alerts",
+    "turnonnotifications": "Turn on notifications",
+    "turnonnotificationsmessage": "Would you like to receive notifications about activities and assignments?",
     "twoparagraphs": "{{p1}}<br><br>{{p2}}",
     "type": "Type",
     "uhoh": "Uh oh!",
diff --git a/src/core/pipes/no-period.ts b/src/core/pipes/no-period.ts
@@ -0,0 +1,39 @@
+// (C) Copyright 2015 Moodle Pty Ltd.
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//     http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
+import { Pipe, PipeTransform } from '@angular/core';
+
+/**
+ * Pipe to remove period at the end of a text if present.
+ */
+@Pipe({
+    name: 'coreNoPeriod',
+})
+export class CoreNoPeriodPipe implements PipeTransform {
+
+    /**
+     * Takes a text and removes ending period.
+     *
+     * @param text The text to treat.
+     * @returns Treated text.
+     */
+    transform(text: string): string {
+        if (!text) {
+            return '';
+        }
+
+        return text.trim().replace(/\.$/, '');
+    }
+
+}
diff --git a/src/core/pipes/pipes.module.ts b/src/core/pipes/pipes.module.ts
diff --git a/src/core/services/local-notifications.ts b/src/core/services/local-notifications.ts
diff --git a/src/theme/theme.base.scss b/src/theme/theme.base.scss
