Loading...

XML

Word

Printable

Type: Bug
Resolution: Unresolved
Priority: Major
Fix Version/s: None
Affects Version/s: DEV backlog
Component/s: Messages add-on
Labels:

Testing Instructions:
Hide

This issue only needs to be tested in 1 device.

Disable this capability at system level for authenticated users: moodle/user:editownmessageprofile: Edit own user messaging profile

In the app, login in the site and go to User menu > Preferences.

Open the Messages preferences. Check that you see a warning saying that you don't have permissions to edit. This message is the one returned by LMS, since this is an edge case we decided this message was enough.

Check that you see the setting to "Send on enter" and you can change this. This setting is local for the app, it doesn't affect LMS, that's why you can change it.

Now go back and open Notifications preferences. Check that you see a warning saying that you don't have permissions to edit. This message is the one returned by LMS, since this is an edge case we decided this message was enough.

In LMS as admin, allow the permission to authenticated user again.

In the app, perform a Pull To Refresh. Check that you see the form and not the error (both in Notifications and Messages preferences).
Show
This issue only needs to be tested in 1 device. Disable this capability at system level for authenticated users: moodle/user:editownmessageprofile: Edit own user messaging profile In the app, login in the site and go to User menu > Preferences. Open the Messages preferences. Check that you see a warning saying that you don't have permissions to edit. This message is the one returned by LMS, since this is an edge case we decided this message was enough. Check that you see the setting to "Send on enter" and you can change this. This setting is local for the app, it doesn't affect LMS, that's why you can change it. Now go back and open Notifications preferences. Check that you see a warning saying that you don't have permissions to edit. This message is the one returned by LMS, since this is an edge case we decided this message was enough. In LMS as admin, allow the permission to authenticated user again. In the app, perform a Pull To Refresh. Check that you see the form and not the error (both in Notifications and Messages preferences).
Pull Main Diff URL:
https://github.com/moodlehq/moodleapp/pull/4137

Sprint:
Moodle Apps - 2024 i3, Moodle Apps - 2024 i3.1, Moodle Apps - 2024 i3.2

Steps to reproduce:

Disable this capability at system level for authenticated users: moodle/user:editownmessageprofile: Edit own user messaging profile
Access the user messages or notifications preferences via the user account menu or via messaging -> settings cog
An exception is displayed

Expected behaviour:

Do not display the preferences link/cog icon or display a better error message

Since we do not have an existing WebService to check whether the user can edit or not its preference, to avoid unnecessary WS calls and being this a very edge case the proposed solution is just to capture the error an display a nice message in the new page that will be opened.

- - Sort By Name
  - Sort By Date
  - Ascending
  - Descending
  - Thumbnails
  - List
  - Download All

MOBILE-4633_app_messages.png
152 kB
14/Aug/24 7:59 PM
MOBILE-4633_app_notifications.png
115 kB
14/Aug/24 7:59 PM
MOBILE-4633_app.png
158 kB
14/Aug/24 7:45 PM
MOBILE-4633_browser.png
32 kB
14/Aug/24 7:43 PM

Testing discovered

MDL-82794 Rename moodle/user:editownmessageprofile to make clear that it affects notifications (and document it)

Open

Assignee:: Dani Palou

Reporter:: Juan Leyva

Integrator:: Alfonso Salces

Tester:: Isabel Renedo Rouco

Participants:: Alfonso Salces, Dani Palou, Isabel Renedo Rouco, Juan Leyva

Votes:: 0 Vote for this issue

Watchers:: 4 Start watching this issue

Created:: 23/Jul/24 4:54 PM

Updated:: 12/Sep/24 6:29 PM

Estimated:

Not Specified

Remaining:

Logged:

17m

Details

Description

Attachments

Attachments

Issue Links

Activity

People

Dates

Time Tracking

Clockify