laamaa / m8c

Cross-platform M8 tracker headless client
Other
380 stars 81 forks source link

check device disconnect with a test write #82

Closed laamaa closed 2 years ago

laamaa commented 2 years ago

Changes the disconnect detection to use a test packet instead of full reset.

@smootalicious could you please test if this works ?

smootalicious commented 2 years ago

Testing now.

On Wed, Jul 6, 2022 at 6:48 AM Jonne Kokkonen @.***> wrote:

Changes the disconnect detection to use a test packet instead of full reset.

@smootalicious https://github.com/smootalicious could you please test if this works ?

You can view, comment on, or merge this pull request online at:

https://github.com/laamaa/m8c/pull/82 Commit Summary

File Changes

(3 files https://github.com/laamaa/m8c/pull/82/files)

Patch Links:

— Reply to this email directly, view it on GitHub https://github.com/laamaa/m8c/pull/82, or unsubscribe https://github.com/notifications/unsubscribe-auth/AZTFTJ5J7LREJQRVXUWX67LVSVW7XANCNFSM52ZPETBA . You are receiving this because you were mentioned.Message ID: @.***>

smootalicious commented 2 years ago

Works great with wait_for_device true or false. No timeout on Help screen.

On Wed, Jul 6, 2022 at 6:48 AM Jonne Kokkonen @.***> wrote:

Changes the disconnect detection to use a test packet instead of full reset.

@smootalicious https://github.com/smootalicious could you please test if this works ?

You can view, comment on, or merge this pull request online at:

https://github.com/laamaa/m8c/pull/82 Commit Summary

File Changes

(3 files https://github.com/laamaa/m8c/pull/82/files)

Patch Links:

— Reply to this email directly, view it on GitHub https://github.com/laamaa/m8c/pull/82, or unsubscribe https://github.com/notifications/unsubscribe-auth/AZTFTJ5J7LREJQRVXUWX67LVSVW7XANCNFSM52ZPETBA . You are receiving this because you were mentioned.Message ID: @.***>

laamaa commented 2 years ago

Thanks for the help!