Details
-
Improvement
-
Status: Closed
-
Blocker
-
Resolution: Fixed
-
3.4.5, 3.5.2, 3.6
-
MOODLE_34_STABLE, MOODLE_35_STABLE, MOODLE_36_STABLE
-
MOODLE_36_STABLE
-
MDL-63897-master -
-
GDPR Followup Sprint 1
Description
We originally planned to allow for an interface to approve and reject individual contexts. This necessitated splitting the request into a Pre-processing stage, and a processing stage, with the pre-processing stage locating everywhere that the user holds data.
This interface never materialised, and now that we have a combination of per-purpose protection, and per-role deletion, this is arguably no longer required. Any rejection of a context for a specific user should be done via a per-role override to the context purpose instead.
As such, we should remove the preprocessing stage. This becomes especially important given MDL-62563, and MDL-62564, where we will see the creation of automated requests.
This pre-processing stage has a number of existing issues, namely it allows for data to become stale between addition and removal.
Removing this stage and jumping straight to the awaiting approval state, and then fetching the list of contexts immediately before they are handled reduces complexity, and the possibility for this stale state. This also alleviates the need for most (but not all) of MDL-62589.
Attachments
Issue Links
- has been marked as being related by
-
MDL-62589 Unable to reset submitted unapproved SARs after plugins are updated / installed
-
- Closed
-