-
Notifications
You must be signed in to change notification settings - Fork 1
Add username property to UsageEntry #86
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
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.
Have tested and verified that both username and computer are coming through as expected. Happy to approve. LGTM 👍
@BHoMBot check compliance |
@adecler to confirm, the following actions are now queued:
|
@BHoMBot check required |
@adecler to confirm, the following actions are now queued:
|
The check |
The check |
@BHoMBot check versioning |
@pawelbaran to confirm, the following actions are now queued:
There are 55 requests in the queue ahead of you. |
@BHoMBot check ready-to-merge |
@adecler to confirm, the following actions are now queued:
|
Issues addressed by this PR
Fixes #85
This adds the capture of the username on top of the computer ID.
Test files
After compiling this code, make sure to use one of our BHoM UIs. This will generate new log files that will be pushed to our database.
To test that this worked, you can connect to the Mongo database to check that the entries are now containing your username.
The simplest Mongo query to check the result is
db.getCollection("UsageData").find({}).sort({_id: -1})
.