search

nextcloud / apps

🚚 This is an archive. Let us know if you want to take over maintainership of any of these apps
11 stars 29 forks source link

Update base.php #31

Closed bjoern86 closed 5 years ago

bjoern86 commented 7 years ago

function storeUser() now checks for @ within $uid and uses string behind @ as usergroup. eg. user1@domain1.tld; user2 @domain1.tld are now automatically members of group domain1.tld, if group does not exist, this group will be created.

pierreozoux commented 7 years ago

Indeed it is a nice feature :) Looks like it doesn't perform a migration of current users, right?

rakshith-ravi commented 6 years ago

This is a really nice feature to have. Can we have this merged before the next NC release? :see_no_evil:

violoncelloCH commented 5 years ago

Hi, thank you for your contribution! Since we moved user_external to it's dedicated repository at https://github.com/nextcloud/user_external (I'm one of the new maintainers) could you please reopen your PR there?

Also: Please sign-off on your commits (just add a line like Signed-off-by: Firstname Lastname <mail@example.com> to every commit). (see https://github.com/nextcloud/server/blob/master/contribute/developer-certificate-of-origin)

jancborchardt commented 5 years ago

@violoncelloCH @bjoern86 @pierreozoux @rakshith-ravi is this meanwhile applied at the new repository? :) If not please do, as we want to archive this repository at some point.

violoncelloCH commented 5 years ago

an attempt to port this to the new repo is in nextcloud/user_external#65 , so I guess we can close this, right @lavdnone ?

lavdnone commented 5 years ago

Great. Thank you.