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

Move kbdpad_mk2 to kbdpad/mk2 and remove clutter #9221

Closed
wants to merge 1 commit into from

Conversation

Erovia
Copy link
Member

@Erovia Erovia commented May 27, 2020

Description

According to the board's official product page, it's "coming soon", so I believe this is not (yet) a breaking change.
@ai03-2725 can comment on this.

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.
  • 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).

@Erovia Erovia requested a review from a team May 27, 2020 11:33
keyboards/kbdfans/kbdpad/mk2/config.h Show resolved Hide resolved
keyboards/kbdfans/kbdpad/mk2/mk2.c Show resolved Hide resolved
@@ -27,17 +27,17 @@
*/
#define LAYOUT_ortho_6x4( \
Copy link
Member

Choose a reason for hiding this comment

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

Also supports numpad_6x4

Copy link
Member Author

Choose a reason for hiding this comment

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

@ai03-2725 could you help with this one?
Which 3 pads are not used with the numpad_6x4 layout on this pcb?

Copy link
Contributor

Choose a reason for hiding this comment

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

image

@ai03-2725
Copy link
Contributor

The first GB for this product ran nearly a year ago, and is already well into its second or third production run

@zvecr zvecr added the breaking_change Changes that need to wait for a version increment label Jul 10, 2020
@tzarc
Copy link
Member

tzarc commented Nov 5, 2021

Needs conflict resolution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking_change Changes that need to wait for a version increment keyboard
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants