Closed kurt-vd closed 8 years ago
Nice code, the only thing that prevents me from merging is the bug appearing here: https://github.com/notandy/ympd/issues/50#issuecomment-97367868.
Nice code, the only thing that prevents me from merging is the bug appearing here: https://github.com/notandy/ympd/issues/50#issuecomment-97367868.
I'll work on that.
The output code is (yet again) ready to pull now.
Hi,
Any news on this pull request ?
I have it working for some time now. I didn't follow up the actual merge.
Kurt
--- Original message ---
Date: Wed, 12 Aug 2015 06:19:52 -0700 From: Awikatchikaen notifications@github.com To: notandy/ympd ympd@noreply.github.com Cc: Kurt Van Dijck github@vandijck-laurijssen.be Subject: Re: [ympd] add support for output enable/disable (#69)
Hi,
Any news on this pull request ?
Reply to this email directly or view it on GitHub: https://github.com/notandy/ympd/pull/69#issuecomment-130297960
I, as many others, want output control with ympd. As C programmer, I could solve this in a few hours. I had a hard time finding the JQuery constructions, I'm not experienced there.
The code works for me.