SUNET / te-canvas

0 stars 1 forks source link

Add warning text that event template is a global setting #15

Closed ernstwi closed 1 year ago

ernstwi commented 2 years ago

Right now event template settings (title, location, description) apply to the entire Canvas instance.

But they are configured in the same UI as course-specific settings (but in a different tab).

This might be confusing to users. Ideally we could create a separate LTI app just to manage the template settings, and only give access to this to Canvas admins, so that teachers do not touch them. Or perhaps we could limit permissions based on LTI "roles" so that only admins have access to this part of the existing app?

Short of this we should at least add a big warning.

pnts-se commented 1 year ago

Will split template config into default and group. Will hide the default tab for everyone that is not a canvas admin. teachers can configure custom template config for course, or fallback to default configured by admin.

pnts-se commented 1 year ago

24 split template config in group and default, but we still do not prevent changing default config based on LTI role.

pnts-se commented 1 year ago

30 closes this issue