maxi07 / Teams-Presence

Displays the current Microsoft Teams presence as LED with a raspberry pi.
MIT License
40 stars 16 forks source link

Add support for Unicorn HAT Mini #7

Closed NiceLucius closed 2 years ago

NiceLucius commented 3 years ago

The Unicorn pHAT is not produced anymore. The successor is the Unicorn HAT mini. Could support be added?

maxi07 commented 3 years ago

Hey @NiceLucius, thanks for reaching out. I have heard about the issue and I am planning to integrate the Unicorn HAT mini into the script, hopefully with auto detection. But this will take some time. If you have any suggestions feel free to start a pull request.

Max

NiceLucius commented 3 years ago

Hi @maxi07. I've started on the support for blinkt but had another major issue. I couldn't retrieve the presence information. After a very long search I discovered that the SCOPES array includes correctly "User.Read" but misses "Presence.Read". Instead, the array includes a "User.ReadBasic.All" which you don't need. Please correct this. You can then also make use of the fact that the graph presence API is not beta anymore so you can use the URL which includes "v1.0". Thanks.

maxi07 commented 3 years ago

Hey @NiceLucius , I made some changes to the code and followed your suggestion. Thank you for noticing!

maxi07 commented 2 years ago

As there does not seem to be any demand for this feature, this will be closed with wontfix. Feel free to submit a pull request!