Uploaded image for project: 'Moodle app'
  1. Moodle app
  2. MOBILE-3805 Moodle App notifications improvements for 4.0
  3. MOBILE-3829

Default fallback for Push Notifications should also display the nofitication message

    XMLWordPrintable

Details

    • Sub-task
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 3.9.4
    • 4.0.0
    • Push notifications
    • Hide

      This issue only needs to be tested in 1 device.

      1. Install the app and login with a user.
      2. Make the user receive a push notification that cannot be opened inside the app. E.g. analytics notifications (Insights generated by prediction models). I don't know if there's an easy way to send those notifications, maybe there's an easier example.
      3. Click the push notification. Check that the notifications list page is opened and, immediately, a new page is opened with the notification details.
      Show
      This issue only needs to be tested in 1 device. Install the app and login with a user. Make the user receive a push notification that cannot be opened inside the app. E.g. analytics notifications (Insights generated by prediction models). I don't know if there's an easy way to send those notifications, maybe there's an easier example. Click the push notification. Check that the notifications list page is opened and, immediately, a new page is opened with the notification details.
    • MOODLE_39_STABLE
    • MOODLE_400_STABLE
    • Moodle App 4.0

    Description

      Our notifications section in the app only displays notifications coming from the notifications popup WS

      So it could happen that you can receive and click to open Push Notifications that currently do not have a handler, in this case, the app won't show anything

      We should  try to do the following:

      • If there is not a delegate for the Push Notification, show the Notifications tab option but also display in a popup the message (we could try to do customdata.extendedtext = message)

      Attachments

        1. screenshot-1.png
          screenshot-1.png
          365 kB
        2. screenshot-2.png
          screenshot-2.png
          204 kB
        3. screenshot-3.png
          screenshot-3.png
          291 kB

        Issue Links

          Activity

            People

              dpalou Dani Palou
              jleyva Juan Leyva
              Noel De Martin Noel De Martin
              Juan Leyva Juan Leyva
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                22/Apr/22