Loading...

XML

Word

Printable

Details

Type: Bug
Resolution: Fixed
Priority: Minor
Fix Version/s: 4.0.4
Affects Version/s: 4.0.2
Component/s: Usability
Labels:
- ci
- patch
- regression
- release_notes
- triaged

Affected Branches:
MOODLE_400_STABLE
Fixed Branches:
MOODLE_400_STABLE
Pull from Repository:
https://github.com/dthies/moodle.git
Pull Main Branch:
~~MDL-74873~~-master
Pull Main Diff URL:
https://github.com/moodle/moodle/compare/master...dthies:MDL-74873-master
Testing Instructions:
Hide

Open preferences in the user menu.

Select Notification preferences from menu.

In upper left check disable notifications.

Verify switch icons for notification types are dimmed and can not be changed.

Reload the page

Verify that these controls are still inactive.

Uncheck disable notifications

Verify that enabled notifications controls are active.

Reload page and confirm they are still active.
Show
Open preferences in the user menu. Select Notification preferences from menu. In upper left check disable notifications. Verify switch icons for notification types are dimmed and can not be changed. Reload the page Verify that these controls are still inactive. Uncheck disable notifications Verify that enabled notifications controls are active. Reload page and confirm they are still active.

Description

If an user goes to the the notification preference page and disables notifications at the top all the controls are disabled on the page, but when the page is reloaded that appear to be enabled. This can cause experienced users to believe notifications are enabled when they are not.

See discussion https://moodle.org/mod/forum/discuss.php?d=434508

Attachments

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

1_MDL-74873_master.png
59 kB
31/Aug/22 4:39 PM
1_MDL-74873_v400.png
56 kB
31/Aug/22 4:39 PM
2_MDL-74873_master.png
59 kB
31/Aug/22 4:39 PM
2_MDL-74873_v400.png
58 kB
31/Aug/22 4:39 PM

Issue Links

is a regression caused by

MDL-67853 Remove online-offline options on notifications.

Closed

Activity

People

Assignee:: Daniel Thies

Reporter:: Daniel Thies

Peer reviewer:: David Mudrák (@mudrd8mz)

Integrator:: Sara Arjona (@sarjona)

Tester:: John Edward Pedregosa

Participants:: CiBoT, Daniel Thies, David Mudrák (@mudrd8mz), John Edward Pedregosa, Sara Arjona (@sarjona)

Votes:: 1 Vote for this issue

Watchers:: 4 Start watching this issue

Dates

Created:: 30/May/22 2:23 AM

Updated:: 05/Sep/22 1:37 PM

Resolved:: 02/Sep/22 1:29 PM

Fix Release Date:: 12/Sep/22

Time Tracking

Estimated:

Remaining:

Logged:

55m