-
Notifications
You must be signed in to change notification settings - Fork 136
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
RUM-8672 Update UserInfo API to make id mandatory. #2195
Conversation
Datadog ReportBranch report: ✅ 0 Failed, 357 Passed, 3459 Skipped, 37.7s Total duration (1m 57.82s time saved) |
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.
LGTM👌
ccb40c7
to
60b18f9
Compare
/merge |
View all feedbacks in Devflow UI.
This merge request is not mergeable yet, because of pending checks/missing approvals. It will be added to the queue as soon as checks pass and/or get approvals.
The median merge time in
|
What and why?
Makes
id
parameter ofsetUserInfo()
mandatory. Old signature is kept with deprecation warning.Review checklist
make api-surface
)