Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Investigate Lessons Learnt from nathankellenicki/node-poweredup #51

Closed
tthiery opened this issue Jul 14, 2020 · 1 comment
Closed

Investigate Lessons Learnt from nathankellenicki/node-poweredup #51

tthiery opened this issue Jul 14, 2020 · 1 comment
Labels
investigation Stuff to investigate
Milestone
v2.1

Comments

@tthiery
Copy link
Member

tthiery commented Jul 14, 2020

Source code at nathankellenicki/node-poweredup under MIT license
@tthiery tthiery added the investigation Stuff to investigate label Jul 14, 2020
@tthiery tthiery added this to the v2.1 milestone Jul 14, 2020
@tthiery
Copy link
Member Author

tthiery commented Jul 15, 2020

Interesting

  • Widest range of supported devices
  • Why is the temperature queried in the beginning of the technic hub connection
  • WeDo 2.0 is a full blown parallel development (WeDoHub->BaseHub ... TechnicMediumHub->LPF2Hub->BaseHub)
  • Devices have some kind of (primary) mode

Do not regress

  • Write direct binary messages in methods of devices / hubs.

Lessons Learnt

  • So many already ;)
  • Make button and other hub properties observable (seen in many libs)
  • Support writing of hub properties

This was referenced Jul 15, 2020
Closed
Closed
Closed
@tthiery tthiery closed this as completed Jul 15, 2020
@tthiery tthiery mentioned this issue Aug 31, 2020
Open
@tthiery tthiery mentioned this issue Dec 23, 2020
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
investigation Stuff to investigate
Projects
None yet
Milestone
v2.1
Development

No branches or pull requests
1 participant
@tthiery