search

sinricpro / esp8266-esp32-sdk

Library for https://sinric.pro - simple way to connect your device to Alexa, Google Home, SmartThings and cloud
https://sinric.pro
Other
236 stars 125 forks source link

Heartbeat not needed? #40

Closed khorner116 closed 4 years ago

khorner116 commented 4 years ago

Things seem to be going fine with my tests of "Pro" and Alexa. I note though that the examples given have dropped the heartbeat code issue that I encountered in the original Sinric implementation. I'm assuming you've taken that into account in your new Pro libraries and are handling it internally. Am I correct?

Ken

kakopappa commented 4 years ago

Yes this is handled in the sdk now

On Thu, 13 Feb 2020 at 10:05 AM khorner116 notifications@github.com wrote:

Things seem to be going fine with my tests of "Pro" and Alexa. I note though that the examples given have dropped the heartbeat code issue that I encountered in the original Sinric implementation. I'm assuming you've taken that into account in your new Pro libraries and are handling it internally. Am I correct?

Ken

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/sinricpro/esp8266-esp32-sdk/issues/40?email_source=notifications&email_token=ABZAZZV4IMRKOFGY4VVL3K3RCS2H7A5CNFSM4KUI4742YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4INEGPNA, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABZAZZXEEGRA7GLRG6IRGCLRCS2H7ANCNFSM4KUI474Q .