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

--ctrlmidich improved documentation #1053

Open
wants to merge 2 commits into
base: next-release
Choose a base branch
from

Conversation

pljones
Copy link
Contributor

@pljones pljones commented Oct 5, 2024

Short description of changes

jamulussoftware/jamulus#3394 adds a new feature to the --ctrlmidich option to allow binding a MIDI CC number explicitly to the participant's "own" channel on the server, regardless of the assigned channel number.

This required some changes to the documentation to explain how it worked. I took the opportunity to clarify (hopefully) the detailed information in Tips & Tricks.

Context: Fixes an issue? Related issues

See jamulussoftware/jamulus#3394

Status of this Pull Request

Looks okay locally.

What is missing until this pull request can be merged?

Needs reviewing and shouldn't be merged until after jamulussoftware/jamulus#3394

Does this need translation?

YES

Checklist

  • I've verified that this Pull Request follows the general code principles
  • I'm sure that this Pull Request goes to the correct branch
  • I waited some time after this Pull Request was opened and all GitHub checks completed without errors.

wiki/en/Tips-Tricks-More.md Outdated Show resolved Hide resolved
wiki/en/Tips-Tricks-More.md Outdated Show resolved Hide resolved
wiki/en/Tips-Tricks-More.md Outdated Show resolved Hide resolved
wiki/en/Tips-Tricks-More.md Show resolved Hide resolved
wiki/en/Tips-Tricks-More.md Outdated Show resolved Hide resolved
@pljones pljones force-pushed the ctrlmidich-improved-doc branch 2 times, most recently from c0c64d5 to 843b811 Compare October 13, 2024 09:54
Copy link
Member

@ann0see ann0see left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On the basis that @ignotus666 also gives his ok to this...

Copy link
Contributor

@ignotus666 ignotus666 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@pljones
Copy link
Contributor Author

pljones commented Oct 14, 2024

Spotted another couple of places I'd said "0 is default" (when it's not -- feature disabled is default).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
Status: Waiting on Team
Development

Successfully merging this pull request may close these issues.

4 participants