Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Command to list custom roles? #26

Closed
zGrav opened this issue Apr 18, 2016 · 6 comments
Closed

Command to list custom roles? #26

zGrav opened this issue Apr 18, 2016 · 6 comments

Comments

@zGrav
Copy link
Contributor

zGrav commented Apr 18, 2016

Hey all,

Is it possible to save a new custom role upon adding it to user (making a new section in hubot's brain for custom roles) and then listing them when asked?

I'll try to make a PR when I get some code up.

Regards.

@zGrav
Copy link
Contributor Author

zGrav commented Apr 18, 2016

Delete custom roles added to #27

@cecilia-sanare
Copy link

I think the meat of what you want is a more structured role system. Where roles are their own entity rather then being linked directly to users.

Can you give a use-case where this would be useful to you?

@zGrav
Copy link
Contributor Author

zGrav commented Apr 19, 2016

Possibly @ceci-woodward , although hubot-auth does do what it is intended to do and works just fine for us.

Our edge case is pretty much we have our Admins in the environment variable of hubot-auth and we've built some scripts (in this case an rss reader) that should only be accessed/used by admins and people in the rss role, hence custom roles.

Regards.

@patcon
Copy link
Member

patcon commented Apr 19, 2016

fwiw, what you're calling "custom roles" is, as I understand it, not an edge case but the main thrust of what hubot-auth is intended to be used for. The "custom" qualifier was confusing me a bit at first tbh :)

@zGrav
Copy link
Contributor Author

zGrav commented Apr 19, 2016

@patcon Sorry, I tend to confuse people with my attempts at doing things properly haha

@zGrav
Copy link
Contributor Author

zGrav commented Apr 19, 2016

Merged, thanks once more!

@zGrav zGrav closed this as completed Apr 19, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

3 participants