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

Checkbox disappears for disabled messaging notification settings (regression)

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide
      1. Login as admin
      2. Click the message drawer icon
      3. Click the cog icon in the message drawer
      4. verify you see a toggle switch before the "Email" notification preference
      5. verify you see a toggle switch before the "Use enter to send" notification preference
      6. verify you can toggle both preferences
      7. Toggle both preferences on
      8. Reload the page
      9. Navigate back to the message drawer prefences
      10. verify the toggles are still on
      11. Toggle both preference off
      12. Navigate back to the message drawer prefences
      13. verify the toggles are still off
      Show
      Login as admin Click the message drawer icon Click the cog icon in the message drawer verify you see a toggle switch before the "Email" notification preference verify you see a toggle switch before the "Use enter to send" notification preference verify you can toggle both preferences Toggle both preferences on Reload the page Navigate back to the message drawer prefences verify the toggles are still on Toggle both preference off Navigate back to the message drawer prefences verify the toggles are still off
    • Affected Branches:
      MOODLE_38_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-66396-master

      Description

      Checkbox for disabled user messaging notification settings is not visible anymore. I think this is a MDL-64887 regression.

      Discovered while running new behat tests built on MDL-65620.

      Steps to reproduce:

      1. As admin check that site messaging system is enabled
      2. Open user's messaging drawer
      3. Click on the cog to see their messaging settings.
      4. Disable "Use enter to send" option
      5. Checkbox for "User enter to send" option is not visible anymore 

       

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                basbrands Bas Brands
                Reporter:
                amaia Amaia Anabitarte
                Peer reviewer:
                Amaia Anabitarte
                Participants:
                Component watchers:
                Jake Dallimore, Jun Pataleta, Bas Brands, Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Sara Arjona (@sarjona), Víctor Déniz Falcón
              • Votes:
                0 Vote for this issue
                Watchers:
                4 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 - 3 hours
                  3h