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

S4 MK3 internal mixer #636

Merged
merged 4 commits into from
May 4, 2024
Merged

Conversation

acolombier
Copy link
Member

Depends on #634

Manual update related to this change

@ronso0
Copy link
Member

ronso0 commented Apr 7, 2024

Copy link
Member

@ronso0 ronso0 left a comment

Choose a reason for hiding this comment

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

Some comments

@acolombier
Copy link
Member Author

@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 :)

@acolombier acolombier changed the title Feat/s3 mk3 internal mixer S4 MK3 internal mixer Apr 8, 2024
@ronso0
Copy link
Member

ronso0 commented Apr 14, 2024

LGTM, do you consider this finished?

I just merged #634 , wanna rebase?

@acolombier
Copy link
Member Author

All ready now!

Copy link
Member

@ronso0 ronso0 left a comment

Choose a reason for hiding this comment

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

some more suggestions

@ronso0
Copy link
Member

ronso0 commented May 4, 2024

Thank you for your patience!

@ronso0 ronso0 merged commit a8a39df into mixxxdj:main May 4, 2024
9 checks passed
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