Closed nbolender closed 3 years ago
Any update on merging this PR? I compiled it, but just wondering since it seems ready to merge and would be nice for others to have it :D
Thanks for the friendly reminder. I'll have some time next week to look over this and I will merge it if all looks good.
Looking forward to this!
I would sincerely appreciate having these features.
I hear you. Apologies for the delay.
Great work. Thanks.
Thanks for this great addition. I wonder if the the -a
flag should support listing the device ID or maybe a new flag. It took me a while to figure out how to actually determine my device ID.
@workmanw it is available when using the CLI or JSON formats. In this PR I had it displaying in the default format (human), but it was later changed to show only the device name.
Maybe this note should at least be added to the readme.
@workmanw it is available when using the CLI or JSON formats. In this PR I had it displaying in the default format (human), but it was later changed to show only the device name.
Maybe this note should at least be added to the readme.
Oh perfect! Yea I didn't realize there were different formats. I think a note in the README would be perfect.
Merges commits from @m3g0byt3 that added support for displaying and setting by device ID.
Also adds support for Device UID. See below for why this is necessary in some situations: