denon-avr icon indicating copy to clipboard operation
denon-avr copied to clipboard

Now we depends on fresh telnet-client v0.7.0.

Open aborovsky opened this issue 9 years ago • 5 comments

Please, help to test my PR in all ways and merge it with your repo.

aborovsky avatar Feb 17 '16 09:02 aborovsky

Hey

I'd love to switch to the more popular telnet-client instead of my own fork. But it isn't really working as I'd like. The biggest problem being the chaining of requests, which your pull request on my fork fixed: https://github.com/phillipsnick/node-telnet-client/pull/9

I have an example here: https://github.com/phillipsnick/denon-avr/blob/chain-requests/examples/telnet/chain.js

Had a little play with telnet-client v0.7.0 but I can't get multiple requests working properly.

Could you maybe put an example file in with it working?

phillipsnick avatar Feb 20 '16 12:02 phillipsnick

I've fixed a lot and tested well.

Now it's working with fresh telnet-client and self-written tiny queue handling inside of /lib/telnet.js method 'handleQueue'

aborovsky avatar Feb 29 '16 16:02 aborovsky

I like the idea, would you be able to adjust the pull request to use 2 space indentation to match the existing code?

I'll then have a proper play and see if it works with my automation system.

phillipsnick avatar Feb 29 '16 23:02 phillipsnick

No problem, I'll fix identation. Вт, 1 марта 2016 г. в 2:56, Nick Phillips [email protected]:

I like the idea, would you be able to adjust the pull request to use 2 space indentation to match the existing code?

I'll then have a proper play and see if it works with my automation system.

— Reply to this email directly or view it on GitHub https://github.com/phillipsnick/denon-avr/pull/12#issuecomment-190455121 .

[image: BeeToo] http://beetoo.me/ Когда умный дом по-настоящему умный Александр Боровский, Технический директор Alexander Borovsky, CTO, Co-Founder +7-926-0-945-00 <+7926094500> +7-499-391-33-62 <+74993913362>[email protected] http://beetoo.me

aborovsky avatar Mar 01 '16 06:03 aborovsky

Identation and other spaces fixed!

aborovsky avatar Mar 01 '16 09:03 aborovsky