-
Notifications
You must be signed in to change notification settings - Fork 1
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
feat: Add RPC sendDeviceInfo() + device-info event #270
Conversation
@gmaclennan is this ready for review? |
No tests yet, but otherwise done. |
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.
changes lgtm so far! can re-review when tests are added
@gmaclennan added a couple of tests in 8fdb7ea if you want to take a second to look through those before merging |
Looks good to me. I can't approve since I own the PR, but ok to merge. |
Co-authored-by: Andrew Chou <andrewchou@fastmail.com>
First part of #245
TODO: tests