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

Add Yaeltex MiniMixxx #456

Merged
merged 11 commits into from
Jan 14, 2024
Merged

Add Yaeltex MiniMixxx #456

merged 11 commits into from
Jan 14, 2024

Conversation

ywwg
Copy link
Member

@ywwg ywwg commented Nov 21, 2021

Ran make gettext, which updated a bunch of other files as well

Ran `make gettext`, which updated a bunch of other files as well
@ywwg ywwg marked this pull request as draft November 21, 2021 02:42
@ywwg ywwg changed the base branch from 2.3 to main November 21, 2021 02:43
@ywwg ywwg marked this pull request as ready for review November 21, 2021 16:32
@ywwg
Copy link
Member Author

ywwg commented Nov 21, 2021

(broken links are not in the new entry)

@Holzhaus
Copy link
Member

Ran make gettext, which updated a bunch of other files as well

Please put these changes into a separate PR IMHO.

@ywwg
Copy link
Member Author

ywwg commented Nov 29, 2021

#458

@Swiftb0y
Copy link
Member

My mistake, shouldn't have merged the controller mapping without the manual entry

@ywwg
Copy link
Member Author

ywwg commented Dec 3, 2021

ok now it just includes the new file

~~~~~~~~~~

The MiniMixxx is a USB class compliant MIDI device. To work correctly with this
mapping, you should load blah blah blah custom layout.
Copy link
Member

Choose a reason for hiding this comment

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

?

Copy link
Member Author

Choose a reason for hiding this comment

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

hahahahhaahhaa sorry. The controller requires a specific config to be loaded on it, I have to attach that

@ywwg ywwg marked this pull request as draft December 3, 2021 17:15
@ywwg ywwg marked this pull request as ready for review December 3, 2021 18:25
@ywwg
Copy link
Member Author

ywwg commented Dec 3, 2021

(I am also working with yaeltex to make this config loaded on controllers by default)

@Swiftb0y
Copy link
Member

Swiftb0y commented Dec 3, 2021

Feel free to ping me when this is ready to review

@ywwg
Copy link
Member Author

ywwg commented Dec 5, 2021

link checker fixed in head. @Swiftb0y ready for review

source/hardware/controllers/yaeltex_minimixxx.rst Outdated Show resolved Hide resolved
source/hardware/controllers/yaeltex_minimixxx.rst Outdated Show resolved Hide resolved
source/hardware/controllers/yaeltex_minimixxx.rst Outdated Show resolved Hide resolved
source/hardware/controllers/yaeltex_minimixxx.rst Outdated Show resolved Hide resolved
@Holzhaus
Copy link
Member

ping

@ywwg
Copy link
Member Author

ywwg commented Feb 10, 2022

oh! I didn't see the comments. I will address

@JoergAtGithub JoergAtGithub added this to the 2.4 milestone Dec 7, 2023
@Holzhaus
Copy link
Member

Holzhaus commented Dec 8, 2023

ping

@JoergAtGithub
Copy link
Member

Hello @ywwg, when will you address the review remarks? We need this manual for 2.4 - without this, we would have to reverse the merge of the mapping PR.

@ywwg
Copy link
Member Author

ywwg commented Jan 6, 2024

sorry for not seeing the pings, github notifications don't work for me (I get literally hundreds per day due to my company's configuration). working on this now

@ywwg ywwg changed the base branch from main to 2.4 January 6, 2024 17:50
@JoergAtGithub
Copy link
Member

Thank you!

@ywwg
Copy link
Member Author

ywwg commented Jan 6, 2024

notes addressed I believe! thank you for your multi-year patience gahhh

:alt: Yaeltex Minimixxx (product image)
:figclass: pretty-figures

Vestax VCI-400 (top view with annotations)
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Vestax VCI-400 (top view with annotations)
Yaeltex Minimixxx (top view with annotations)

Copy link
Member Author

Choose a reason for hiding this comment

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

heh oops

Mapping description
-------------------

.. figure:: ../../_static/controllers/yaeltex_minimixxx.jpg
Copy link
Member

Choose a reason for hiding this comment

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

Do we have a permission to use this photo, or is it your own work?

Copy link
Member Author

Choose a reason for hiding this comment

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

I was working with the manufacturer to design the controller and they are ok with this.

@Swiftb0y Swiftb0y dismissed their stale review January 7, 2024 17:37

Thanks for addressing my previous comments. Unfortunately I'm unable to do a proper review right now.

@JoergAtGithub
Copy link
Member

LGTM! Thank you!

@JoergAtGithub JoergAtGithub merged commit 26d6bcd into mixxxdj:2.4 Jan 14, 2024
8 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants