hharnisc / meteor-accounts-admin-ui-bootstrap-3

A roles based account management system using bootstrap 3 for Meteor
MIT License
56 stars 55 forks source link

Made a very basic 'Add User' dialog box #15

Closed clarkritchie closed 10 years ago

clarkritchie commented 10 years ago

This is quite primitive, but I needed something functional to move forward.

hharnisc commented 10 years ago

Tried it out and it's not quite ready to be pushed to master.

Here's some notes:

Add User Dialog adduser

Add User Button

adduserbutton

Another thing to think about, this only seems to cover the meteor base accounts package. And doesn't work where all users a facebook/twitter/G+ etc.

hharnisc commented 10 years ago

I'm going to close this for now. When you've got some more changes I'll pull it and test it again.

clarkritchie commented 10 years ago

No worries, and thanks for the feedback. How would you approach the use case for apps that don't support external authentication and don't permit self registration? We don't do either, and thus need the standard functionality like adding users manually and resetting passwords. Maybe that's a module within this that someone can optionally enable if they need to also do local user management as well as role management?

hharnisc commented 10 years ago

A couple of cases

"accounts-password" installed in list of packages

If the "accounts-password" package is not installed I would hide the "Add User" button.

With mixed users it seems like it would be fine to have the option to add users so long as it was clear that the user type would be "accounts-password".

I'm sure there's some more cases here that need to be though through, but that might be good start. I think this should at least cover your case, as well as the case with mixed accounts. I'll see if I can find anyone with a good way of detecting if packages are installed.