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

Numark DJ2Go re-enable note-off for deck A cue button #2087

Merged
merged 1 commit into from
Apr 24, 2019
Merged

Numark DJ2Go re-enable note-off for deck A cue button #2087

merged 1 commit into from
Apr 24, 2019

Conversation

mrvanes
Copy link
Contributor

@mrvanes mrvanes commented Apr 23, 2019

I don't know why the note-off for deck A cue button was disabled, but it breaks the hold-cue functionality of Mixxx cue setting and moreover, deck B has both note-on and note-off (and behaves normally) so the configuration was asymmetric anyway?

I don't know why the note-off for deck A cue button was disabled, but it breaks the hold-cue functionality of Mixxx cue setting and moreover, deck B has both note-on and note-off (and behaves normally) so the configuration was asymmetric anyway?
@ronso0
Copy link
Member

ronso0 commented Apr 23, 2019

thanks for the fix.
please prefix the PR with "Numark DJ2Go" so it's clear what this is about. thx!

@mrvanes mrvanes changed the title Re-enable note-off for deck A cue button Numark DJ2Go re-enable note-off for deck A cue button Apr 24, 2019
@uklotzde
Copy link
Contributor

Someone must have forgot to remove those comments after testing!

LGTM.

@uklotzde uklotzde merged commit 6487ad5 into mixxxdj:2.2 Apr 24, 2019
@uklotzde uklotzde added this to the 2.2.2 milestone Apr 24, 2019
@mrvanes mrvanes deleted the patch-2 branch April 25, 2019 05:37
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