-
Notifications
You must be signed in to change notification settings - Fork 13.1k
fix: users.update API call erases other customFields #36616
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
Conversation
Co-authored-by: Jessica Schelly Souza <jessica_schelly@hotmail.com>
🦋 Changeset detectedLatest commit: 2e3b22d The changes in this PR will be included in the next version bump. This PR includes changesets to release 35 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
Looks like this PR is not ready to merge, because of the following issues:
Please fix the issues and try again If you have any trouble, please check the PR guidelines |
cardoso
left a comment
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.
All good if CI passes
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## release-7.4.5 #36616 +/- ##
==============================================
Coverage 59.47% 59.47%
==============================================
Files 2830 2830
Lines 68570 68570
Branches 15175 15175
==============================================
Hits 40781 40781
Misses 25120 25120
Partials 2669 2669
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
|
cardoso
left a comment
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.
Updated SAML dockerfile
Backport of #36578