Added close method - Githubissues

geut / discovery-swarm-webrtc

discovery-swarm for WebRTC

MIT License

94 stars 23 forks source link

Closed RangerMauve closed 5 years ago

RangerMauve commented 5 years ago

As per #1 I added a close method, that's aliased under destroy to be similar to discovery-swarm.

Lemme test it out with dat-js before you merge it though. 😅

tinchoz49 commented 5 years ago

Thanks @RangerMauve ! tell me how was that test in dat-js and we can merge it :+1:

RangerMauve commented 5 years ago

K, it's good to go. Connections are being closed, and there are no weird errors. 😀

I'm finding that it takes a few seconds before peers are actually connected, but I'll probably look into that more next week.

tinchoz49 commented 5 years ago

thanks @RangerMauve !

RangerMauve commented 5 years ago

Would you mind pinging me when it's released on NPM so I can update the dependencies inside dat-js? 😄

tinchoz49 commented 5 years ago

published!