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

Migrate communication provider settings logic from course edit into its own location

XMLWordPrintable

      When we moved the communication provider settings out of the course settings into their own page, the UI was moved, but the logic was all kept within the course settings. One of the reasons for this was that a couple of course changes require provider updates (namely course avatar changes and switching category to hidden), but we should just handle those properly and move everything else out - either by keeping those parts of logic in there (they only need to happen if the course settings change, and don't need to also be copied out into the communication updates), or possibly we could look into hooks for those.

      The reason the current placement of this logic is not ideal is:

      1. It requires hacking the data when you actually save course setting, because it has to pull in the communication data to avoid overwriting it while saving course settings.
      2. The separate communication page is designed to be used for whichever context any current/future implementation of communication is used, so it may not only relate to courses in future.
      3. We're saving course settings unnecessarily when saving provider settings.

            Unassigned Unassigned
            michaelh Michael Hawkins
            Votes:
            0 Vote for this issue
            Watchers:
            8 Start watching this issue

              Created:
              Updated:

                Error rendering 'clockify-timesheets-time-tracking-reports:timer-sidebar'. Please contact your Jira administrators.