- Notifications
You must be signed in to change notification settings - Fork1k
feat: add groups and group members to telemetry snapshot#13655
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
austinrhode commentedJun 24, 2024
- Updated Telemetry data to also send a list of groups and a list of group members
…gic to ignore non-active users when creating a snapshot
github-actionsbot commentedJun 24, 2024 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
All contributors have signed the CLA ✍️ ✅ |
| for_,dbUser:=rangeusers { | ||
| ifdbUser.Status!=database.UserStatusActive { | ||
| continue | ||
| } |
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.
It looks like this is from your previous pr#13613
coderd/telemetry/telemetry.go Outdated
| } | ||
| } | ||
| funcConvertGroupMember(group_member database.GroupMember)GroupMember { |
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.
Don't use underscores in go vars.
| funcConvertGroupMember(group_member database.GroupMember)GroupMember { | |
| funcConvertGroupMember(member database.GroupMember)GroupMember { |
I have read the CLA Document and I hereby sign the CLA |