Skip to content

Comments

Settings UI changes for ASIO#999

Closed
mulyaj wants to merge 29 commits intojamulussoftware:MoveASIOfrom
mulyaj:ASIO_UI
Closed

Settings UI changes for ASIO#999
mulyaj wants to merge 29 commits intojamulussoftware:MoveASIOfrom
mulyaj:ASIO_UI

Conversation

@mulyaj
Copy link
Contributor

@mulyaj mulyaj commented Feb 14, 2021

Renamed "ASIO Setup" to "ASIO Device Settings" and created a QTgroupbox for devices.

ann0see and others added 29 commits February 2, 2021 19:44
New function SetMixerBoardDeco sets title color and background according
to recording state and skin.
Called on recorder state change, skin change and disconnect
…questTemplate

Add feature request template
…odules-to-ssh

Move submodules to relative paths
…server-recording

feature-req925 recording indicator
The relative paths in .gitmodules contained a stray github.com/
path component which broke submodule resolution.

Got broken in jamulussoftware#934
Fixes jamulussoftware#984

Signed-off-by: Christian Hoffmann <[email protected]>
Fix git submodule usage (which I - pljones - broke...)
Embedded quotes were included in strings by doubling them: `""`
This doesn't work in C++, so changed to `\"`
Also warn when this option is used when not running in server mode
or when not registering to a central server.

Signed-off-by: Christian Hoffmann <[email protected]>
…icip-logging

Add logging for --serverpublicip
@mulyaj mulyaj closed this Feb 14, 2021
@mulyaj mulyaj reopened this Feb 14, 2021
@ann0see
Copy link
Member

ann0see commented Feb 14, 2021

@mulyaj I think you need to open this PR based on the MoveASIO branch not on master.

@mulyaj
Copy link
Contributor Author

mulyaj commented Feb 14, 2021

Yeah I did something wrong here... can you remove this?

@mulyaj mulyaj closed this Feb 14, 2021
@ann0see
Copy link
Member

ann0see commented Feb 14, 2021

Wait a moment, I'll try to do a git cherry-pick here.

@mulyaj
Copy link
Contributor Author

mulyaj commented Feb 14, 2021

I'm not sure how to just get the .UI and .cpp changes I made to PR.

@ann0see
Copy link
Member

ann0see commented Feb 14, 2021

This happens because these changes are based on master and the MoveASIO branch is not up to date with master. Let me check it.

@mulyaj
Copy link
Contributor Author

mulyaj commented Feb 14, 2021

Oh I merged master accidentally, I see. Thanks.

@ann0see
Copy link
Member

ann0see commented Feb 14, 2021

See #1000

@mulyaj
Copy link
Contributor Author

mulyaj commented Feb 14, 2021

Cool! Appreciate it.

@ann0see
Copy link
Member

ann0see commented Feb 14, 2021

Just merged it. I assume this is your first PR here? Thank you very much ;-)!

You should add yourself to the contributor list. I'll link to it shortly.

@ann0see
Copy link
Member

ann0see commented Feb 14, 2021

Ok. You should add yourself to this file now: https://github.com/jamulussoftware/jamulus/blob/master/src/util.cpp

Just add the main repo via

git remote add upstream [email protected]:jamulussoftware/jamulus

git fetch upstream
git checkout upstream/master (I hope this is correct)
git checkout -b "AddContributor"

Then do your changes, push it and open a PR here.

@mulyaj
Copy link
Contributor Author

mulyaj commented Feb 14, 2021

Ok, will do

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.

7 participants