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

Profile: Should indicate user email address settings on own profile

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide

      Covered by unit tests.

      For manual tests, following steps might be followed:

      1. Log in as any user.
      2. Go to your profile page.
      3. Go to edit profile page.
      4. Set "Email display" setting to "Allow everyone to see my email address"
      5. Press update profile
      6. Verify that "Visible to everyone" text appear next to email address.
      7. Go to edit profile page.
      8. Set "Email display" setting to "Allow only other course members to see my email address"
      9. Press update profile
      10. Verify that "Visible to other course participants" text appear next to email address.
      11. Go to edit profile page.
      12. Set "Email display" setting to "Hide my email address from non-privileged users"
      13. Press update profile
      14. Verify that "Hidden from all non-privileged users" text appear next to email address.
      Show
      Covered by unit tests. For manual tests, following steps might be followed: Log in as any user. Go to your profile page. Go to edit profile page. Set "Email display" setting to "Allow everyone to see my email address" Press update profile Verify that "Visible to everyone" text appear next to email address. Go to edit profile page. Set "Email display" setting to "Allow only other course members to see my email address" Press update profile Verify that "Visible to other course participants" text appear next to email address. Go to edit profile page. Set "Email display" setting to "Hide my email address from non-privileged users" Press update profile Verify that "Hidden from all non-privileged users" text appear next to email address.
    • Affected Branches:
      MOODLE_310_STABLE
    • Fixed Branches:
      MOODLE_311_STABLE
    • Pull 3.10 Branch:
      MDL-70439-310
    • Pull 3.11 Branch:
      MDL-70439-311
    • Pull Master Branch:
      MDL-70439-master

      Description

      We had student feedback that they were concerned as to whether the email setting 'Hide my email address from non-privileged users' was actually working. The reason is that on the profile screen it shows your email address, but it doesn't show that setting, so users incorrectly think that the address is visible to other people on their profile.

      To solve this problem I suggest we change it so that:

      • When you are viewing your own profile
      • Or when you have the capability that lets you see all users email addresses (e.g. admins)

      Then next to the email address we add one of the following texts, in brackets, corresponding to the maildisplay option shown:

      • visible to everyone (1)
      • visible to other course members (2)
      • hidden from all non-privileged users (0)

      See attached screenshot

      (Wording changes welcome - I tried to use similar wording to the options in the maildisplay option, but for the last confusing one, I also used the word 'hidden' instead of 'visible' to emphasise the main point.)

      Note I did consider using the exact texts from the maildisplay settings dropdown but those are long and read a bit weird in this context so I think we may need new ones. These are:

      • Hide my email address from non-privileged users
      • Allow everyone to see my email address
      • Allow only other course members to see my email address

      Maybe we should use AMOS to copy the values from the existing string by default, if we can do that, to help translations.

      We will assign an OU developer to code this improvement, but I'm creating this MDL in advance.

        Attachments

          Activity

            People

            Assignee:
            dinhhien0307 DinhPhamVinhHien
            Reporter:
            quen Sam Marshall
            Peer reviewer:
            Sam Marshall
            Integrator:
            Sara Arjona (@sarjona)
            Tester:
            CiBoT
            Participants:
            Component watchers:
            Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Ilya Tregubov, Sara Arjona (@sarjona)
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              10/May/21

                Time Tracking

                Estimated:
                Original Estimate - 0 minutes
                0m
                Remaining:
                Remaining Estimate - 0 minutes
                0m
                Logged:
                Time Spent - 30 minutes
                30m