michaschwab / booc3

https://booc.io
Other
4 stars 1 forks source link

Roles: Actually make them work. #184

Open jamestompkin opened 8 years ago

michaschwab commented 8 years ago

bd7220bf819ddc227213bb953fc26021faec4723 and 07726ff0bb541350facbb07ba272f200d300917d are working toward fixing this issue.

michaschwab commented 8 years ago

i did some more work today and think that TA rights are now fine. i'll change GK's teams' roles to those.

jamestompkin commented 8 years ago

We should run through this together before you do that.

On 2 February 2016 at 15:22, Michail Schwab notifications@github.com wrote:

i did some more work today and think that TA rights are now fine. i'll change GK's teams' roles to those.

— Reply to this email directly or view it on GitHub https://github.com/michaschwab/booc3/issues/184#issuecomment-178801975.

michaschwab commented 8 years ago

alright, you'll be our test case then :-) i revoked your admin privileges, but made you teacher of gov 2001 and james test course. let me know if everything works :-D

michaschwab commented 8 years ago

since this commit af6758ab8972fd2ed5b38baa5c0f95ea5ea162cd content editors can also access the admin panel, but only the functions relating to the material, not the concept-changing things.

jamestompkin commented 8 years ago

Teacher role

I have course edit privileges on 'Gov 2001' and 'James test course'

Courses page:

Materials page:

A course:

Minor thing: The visual 'bar' under which the 'options' normally sit in the bottom right-hand side still appears for me even if I don't have edit privileges.

On 2 February 2016 at 19:09, Michail Schwab notifications@github.com wrote:

since this commit af6758a https://github.com/michaschwab/booc3/commit/af6758ab8972fd2ed5b38baa5c0f95ea5ea162cd content editors can also access the admin panel, but only the functions relating to the material, not the concept-changing things.

— Reply to this email directly or view it on GitHub https://github.com/michaschwab/booc3/issues/184#issuecomment-178900935.

michaschwab commented 8 years ago
jamestompkin commented 8 years ago

This is too complex for me to talk about off the top of my head; can you try and write out the cases and diagram the possible interactions?

On 3 February 2016 at 19:59, Michail Schwab notifications@github.com wrote:

  • New Course creation should now work as teacher

    Course Delete Undo should be fixed now

    Backups: I should limit the access to the correct courses there.

  • Material: yeah you currently have access to all materials. i'm not sure what's the best thing to do here. limiting access to only materials that are assigned to the course you have access to will
    • cause problems with materials that you just created without assigning them to anything?
    • disallow you from embedding contents from other courses into yours. we could just limit the association to concepts to concepts of the courses you have access to?

— Reply to this email directly or view it on GitHub https://github.com/michaschwab/booc3/issues/184#issuecomment-179557559.