-
Notifications
You must be signed in to change notification settings - Fork 125
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
S4MK3: update setting change instruction #634
Conversation
420d3e0
to
fc08e5e
Compare
fc08e5e
to
33545f0
Compare
33545f0
to
b279c70
Compare
source/hardware/controllers/native_instruments_traktor_kontrol_s4_mk3.rst
Outdated
Show resolved
Hide resolved
@daschuer should this be targeted at main, or at manual 2.5? I am unclear how the manual PRs work |
@ronso0 do you know the answer? |
4c7d535
to
211807a
Compare
main is correct here, we don't have a 2.5 branch. |
f65c715
to
211807a
Compare
source/hardware/controllers/native_instruments_traktor_kontrol_s4_mk3.rst
Outdated
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.
LGTM, only the points I have commented on could be improved a little
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.
LGTM, thank you for this improvement!
This PR goes together with this one for S4MK3 settings