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

[keyboard]add XVX M67 keyboard. #24824

Closed
wants to merge 5 commits into from
Closed

Conversation

jonylee1986
Copy link
Contributor

add XVX M67 keyboard.

Description

Types of Changes

  • Core
  • Bugfix
  • New feature
  • Enhancement/optimization
  • Keyboard (addition or update)
  • Keymap/layout/userspace (addition or update)
  • Documentation

Issues Fixed or Closed by This PR

Checklist

  • My code follows the code style of this project: C, Python
  • I have read the PR Checklist document and have made the appropriate changes.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • I have tested the changes and verified that they work and don't break anything (as well as I can manage).

@zvecr
Copy link
Member

zvecr commented Jan 17, 2025

image

I think we are going to require the code for the "wireless" version for validation, before we can consider this PR.

@tzarc tzarc marked this pull request as draft January 17, 2025 08:04
Update product link!
@jonylee1986
Copy link
Contributor Author

@zvecr hello!
We have a pure wired product, the product link has been updated, please let us know if it is feasible!

@zvecr
Copy link
Member

zvecr commented Jan 18, 2025

Per #24085,

If QMK identifies any licensing violation, all current and future PRs from that manufacturer will be put on hold until sources are provided. QMK's license requires full disclosure of source code for any firmware which is based on QMK, includes any part of QMK, or derives from QMK in any way. This source code disclosure is not negotiable and is automatically agreed to by any designer when they choose to use QMK.

Nope, we are going to require the code for the "wireless" version for validation, before we can consider this PR.

@badmark
Copy link

badmark commented Jan 18, 2025

Nope, we are going to require the code for the "wireless" version for validation, before we can consider this PR.

This is a wired only keyboard, what's the issue here?

@adophoxia
Copy link
Contributor

adophoxia commented Jan 18, 2025

Nope, we are going to require the code for the "wireless" version for validation, before we can consider this PR.

This is a wired only keyboard, what's the issue here?

The issue is that for certain manus' that make a wired and wireless variant of a board, if they wanted to have VIA compatibility for their wireless board and tout it so, but not submit the source code for it for petty reasons, they would bypass this via submitting source code for the wired variant, add the keymap/JSON for VIA, to where the wireless variant can piggyback off of the VID and/or PID for auto-detect with VIA.

This is how deceitful companies like CIDOO/Zuoyu, Epomaker, and the like do so for their boards since no matter what, their stock firmware always shows up as a GMMK Numpad in VIA until after a JSON is loaded.

So in this case, if the wireless variant of this xvx m67 does use QMK/VIA, they need to disclose the source code for all variants. In short, release for all, or not and continue this stupid workaround.

@badmark
Copy link

badmark commented Jan 18, 2025

There is no wireless variant of this keyboard!

@fauxpark
Copy link
Member

fauxpark commented Jan 18, 2025

https://www.xvxchannel.com/en-au/collections/xvx-m67-jadestone/products/xvx-m67-jadestone-mechanical-keyboard-with-imd-tech-keycaps-interstellar-black
And you can see in the image above posted by zvecr, there is most definitely a wireless variant.

@zvecr
Copy link
Member

zvecr commented Jan 18, 2025

Posting this as its likely they will try and remove the listing from the site
image

@badmark
Copy link

badmark commented Jan 18, 2025

I have seen this, I am trying to help them do QMK the right way, this was their marketing department's mistake. If they correct this image, can they continue the PR process?

@daskygit
Copy link
Member

I can buy a wireless XVX M67 from Amazon that advertises QMK/VIA support. What exactly is the mistake?

@qmk qmk locked as too heated and limited conversation to collaborators Jan 18, 2025
@tzarc
Copy link
Member

tzarc commented Jan 18, 2025

Locked this issue as it’s devolving — there’s ample evidence and ample ambiguity to demand the wireless variant’s sources.

See license violations - PR considerations for QMK's stance on such things. All PRs for this vendor are now indefinitely on hold accordingly.

Will update the license violations page to reflect the outcome of this PR.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants