-
Notifications
You must be signed in to change notification settings - Fork 119
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
S4 MK3 internal mixer #636
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some comments
source/hardware/controllers/native_instruments_traktor_kontrol_s4_mk3.rst
Outdated
Show resolved
Hide resolved
source/hardware/controllers/native_instruments_traktor_kontrol_s4_mk3.rst
Outdated
Show resolved
Hide resolved
source/hardware/controllers/native_instruments_traktor_kontrol_s4_mk3.rst
Outdated
Show resolved
Hide resolved
source/hardware/controllers/native_instruments_traktor_kontrol_s4_mk3.rst
Outdated
Show resolved
Hide resolved
@ronso0 if you have a chance, could you please help me getting the dependent PR merged? That should ease the work on that one and prevent headachy conflicts :) |
7038ec7
to
bd0f89a
Compare
LGTM, do you consider this finished? I just merged #634 , wanna rebase? |
bd0f89a
to
ec8e1c3
Compare
All ready now! |
source/hardware/controllers/native_instruments_traktor_kontrol_s4_mk3.rst
Show resolved
Hide resolved
source/hardware/controllers/native_instruments_traktor_kontrol_s4_mk3.rst
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
some more suggestions
source/hardware/controllers/native_instruments_traktor_kontrol_s4_mk3.rst
Outdated
Show resolved
Hide resolved
Co-authored-by: ronso0 <[email protected]>
source/hardware/controllers/native_instruments_traktor_kontrol_s4_mk3.rst
Outdated
Show resolved
Hide resolved
Co-authored-by: ronso0 <[email protected]>
Thank you for your patience! |
Depends on #634
Manual update related to this change