search

NottingHack / hms

Hackspace Management System
4 stars 7 forks source link

Provide a view of member's tool induction status for admin purposes #110

Open alex-wells opened 8 years ago

alex-wells commented 8 years ago

It's been suggested that it may be useful for teams responsible for tool maintenance/inductions to have access to the list of inducted members for those tools; it should be a fairly trivial matter to add a view that renders the contents of the tl_members_tools table for individual tools that is accessible only to the inductors, maintainers for that tool and the admins/membership team.

Thoughts?

daniel1111 commented 8 years ago

I think this is a very good idea. It might also be nice to allow maintainers/admins to promote inducted members to inductor and/or maintainer in the system, as well as removing access from previously inducted members (ok, so that last part is probably unnecessary, as I don't think we've ever needed to do that).

dpslwk commented 8 years ago

Actually ability for maintainers to remove access would be good, some one abuses a tool they loose access and need to have a refresher induction.

'RepRap' Matt

On 2 Nov 2015, at 08:47, D notifications@github.com wrote:

I think this is a very good idea. It might also be nice to allow maintainers/admins to promote inducted members to inductor and/or maintainer in the system, as well as removing access from previously inducted members (ok, so that last part is probably unnecessary, as I don't think we've ever needed to do that).

— Reply to this email directly or view it on GitHub.

alex-wells commented 8 years ago

The un-induct feature sounds like a good idea to me; I'm quite happy to pick this up once I've finished with the rest of bouncer, if no-one has any problems with me doing so?

geeksareforlife commented 8 years ago

Sounds good, but it should be maintainers (not inductors) that can un-induct

alex-wells commented 8 years ago

do we want to include trustees/membership admins in the list of those able to do the un-induct?

geeksareforlife commented 8 years ago

This just shows up the issue we have with permissions as HMS gets bigger

It might be better to deal with that issue first, before we add the promote/un-induct parts of this

daniel1111 commented 8 years ago

I'd say probably not membership admin's but maybe those in the 'full access' group should be able to? (if not limited to just maintainers)

dpslwk commented 8 years ago

yeah full access, and leave the rest till we sort ACL out

‘RepRap’ Matt

On 2 Nov 2015, at 10:20, D notifications@github.com wrote:

I'd say probably not membership admin's but maybe those in the 'full access' group should be able to? (if not limited to just maintainers)

— Reply to this email directly or view it on GitHub https://github.com/NottingHack/hms/issues/110#issuecomment-152978819.

daniel1111 commented 8 years ago

But yes - what @geeksareforlife said: permissions in HMS aren't that great atm