Closed sidharthb0 closed 6 months ago
This sounds great. Need to see when I can make some time for testing it though, but do remind me if I take too long!
Thanks!
I really want to add this PR, but have some questions:
Currently you import the threading
module and create a done_event
object, but this object is never used/checked, and the sleep(60)
is also still in the code. Could you have forgotten to add some other changes to the PR?
The script has been modified to include a
threading.Event
object and to wait for this event before disconnecting, rather than using asleep(60)
call. This change should ensure that the script remains active until all operations have completed, without relying on an arbitrary timeout.