-
Notifications
You must be signed in to change notification settings - Fork 393
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
Implicitly force import manually added user #1048
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1048 +/- ##
==========================================
+ Coverage 90.80% 90.82% +0.01%
==========================================
Files 109 109
Lines 9408 9425 +17
==========================================
+ Hits 8543 8560 +17
Misses 513 513
Partials 352 352
|
This pull request introduces 1 alert when merging 11fa663 into 9bd9112 - view on LGTM.com new alerts:
|
This pull request introduces 1 alert when merging 19832e1 into 9bd9112 - view on LGTM.com new alerts:
|
Should we support the same for groups? It would help when switching from SCIM connector… |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.look good
Add
force
parameter todatabricks_user
, so that SCIM sync corner cases might be simplified