Skip to content

Conversation

@jgribonvald
Copy link
Contributor

@jgribonvald jgribonvald commented Apr 3, 2020

Closes #17153 and feature request

As simple as it is, this feature permit to make optional the user creation when a user come from a CAS authentication. The use case is that the user is imported before from on other source and so should exist. It permit also to filter user access.

@CLAassistant
Copy link

CLAassistant commented Apr 3, 2020

CLA assistant check
All committers have signed the CLA.

@jgribonvald
Copy link
Contributor Author

Is there a specific way to provide more i18n translations or is it good like that ?

@jgribonvald jgribonvald force-pushed the feat_CAS_User_Creation_option branch 2 times, most recently from 00d9b59 to fb0b71b Compare April 6, 2020 12:26
@jgribonvald
Copy link
Contributor Author

Hi,
what do you need more to accept this Pull Request ?
Best regards.

@jgribonvald
Copy link
Contributor Author

A little refresh on this PR! What do you think about it ?

@bseclier
Copy link

bseclier commented Jul 9, 2020

Agreed, it would be very useful !

@pierre-lehnen-rc pierre-lehnen-rc self-assigned this Jul 13, 2020
@pierre-lehnen-rc pierre-lehnen-rc added this to the 3.5.0 milestone Jul 13, 2020
@jgribonvald jgribonvald force-pushed the feat_CAS_User_Creation_option branch from f92d326 to 0f1acbe Compare July 21, 2020 06:46
@jgribonvald
Copy link
Contributor Author

All requested change were applied !

@jgribonvald
Copy link
Contributor Author

Is this PR could be merged now ? as it pass all test and it was approved ?

@pierre-lehnen-rc
Copy link
Contributor

Sorry we somehow missed it when cutting the release. I'll make sure it gets merged as soon as the current feature freeze is over.

@pierre-lehnen-rc pierre-lehnen-rc modified the milestones: 3.5.0, 3.8.0 Oct 9, 2020
@sampaiodiego sampaiodiego merged commit 9716730 into RocketChat:develop Oct 13, 2020
@sampaiodiego sampaiodiego mentioned this pull request Nov 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CAS user creation as optional

5 participants