Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add support for output enable/disable #69

Merged
merged 6 commits into from
Sep 20, 2015
Merged

Conversation

kurt-vd
Copy link

@kurt-vd kurt-vd commented Apr 28, 2015

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.

@notandy
Copy link
Owner

notandy commented May 1, 2015

Nice code, the only thing that prevents me from merging is the bug appearing here: #50 (comment).

@kurt-vd
Copy link
Author

kurt-vd commented May 1, 2015

Nice code, the only thing that prevents me from merging is the bug appearing here: #50 (comment).

I'll work on that.

@kurt-vd
Copy link
Author

kurt-vd commented May 2, 2015

The output code is (yet again) ready to pull now.

@awikatchikaen
Copy link

Hi,

Any news on this pull request ?

@kurt-vd
Copy link
Author

kurt-vd commented Aug 16, 2015

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:
#69 (comment)

notandy added a commit that referenced this pull request Sep 20, 2015
add support for output enable/disable
@notandy notandy merged commit 30b0b81 into notandy:master Sep 20, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants