Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-72143

Add an additional check for $CFG->noemailever in the Airnotifier check configuration report

    XMLWordPrintable

Details

    • Improvement
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 3.11.1
    • 4.0
    • Other
    • MOODLE_311_STABLE
    • MOODLE_400_STABLE
    • MDL-72143-master
    • Hide
      1. As site admin go to Site administration -> Notifications > Mobile (you might need to enable Mobile notifications if they were not enabled on your site)
      2. At the bottom of the page click on the "Check and test push notification configuration" link
      3. Confirm that:
        • The check "$CFG->noemailever disabled" says OK
      4. Now, edit your config.json and add the following just before the require_once(): $CFG->noemailever = true;
      5. Open the same page with the checks again and confirm that:
        • The check "$CFG->noemailever disabled" says Critical
        • In the last column of the previous check, you see instructions to disable the $CFG->noemailever setting
      Show
      As site admin go to Site administration -> Notifications > Mobile (you might need to enable Mobile notifications if they were not enabled on your site) At the bottom of the page click on the "Check and test push notification configuration" link Confirm that: The check "$CFG->noemailever disabled" says OK Now, edit your config.json and add the following just before the require_once(): $CFG->noemailever = true; Open the same page with the checks again and confirm that: The check "$CFG->noemailever disabled" says Critical In the last column of the previous check, you see instructions to disable the $CFG->noemailever setting

    Description

      If $CFG->noemailever is set to true, mobile push notifications won't work, we need to add an additional check for it at the beginning of the list:

       

      Attachments

        Activity

          People

            jleyva Juan Leyva
            jleyva Juan Leyva
            Pau Ferrer Pau Ferrer
            Andrew Lyons Andrew Lyons
            Gladys Basiana Gladys Basiana
            Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias, Sujith Haridasan
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Time Tracking

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0 minutes
                0m
                Logged:
                Time Spent - 1 hour, 57 minutes
                1h 57m