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 encoder map support for GMMK Pro #18600

Merged
merged 2 commits into from
Oct 5, 2022
Merged

Conversation

adophoxia
Copy link
Contributor

@adophoxia adophoxia commented Oct 5, 2022

Description

  • Add encoder mapping to rev1 and rev2 variants (VIA keymap for Rev2 ISO already has it)
  • Add rules.mk file to default keymap folders to utilize encoder mapping

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

adophoxia and others added 2 commits October 4, 2022 19:58
- Add encoder mapping to rev1 and rev2 variants (VIA keymap for ISO already has it)
- Add `rules.mk` file to `default` keymap folders to utilize encoder mapping
@github-actions github-actions bot added keymap via Adds via keymap and/or updates keyboard for via support labels Oct 5, 2022
@drashna drashna requested a review from a team October 5, 2022 17:29
@drashna drashna merged commit fe6f707 into qmk:master Oct 5, 2022
@adophoxia adophoxia deleted the gmmk-pro-encoder branch October 5, 2022 17:43
@adophoxia
Copy link
Contributor Author

Ty for merging, @drashna 👍

SjB added a commit to SjB/qmk_firmware that referenced this pull request Oct 12, 2022
* master: (147 commits)
  [Keyboard] Add waffling80 Rev B (qmk#18603)
  Enable mouse keys for momokai tap_trio (qmk#18614)
  [Keyboard] adafruit/macropad: Fix typo (qmk#18616)
  Product changes to 1up lineup (qmk#18382)
  width:2 → height:2 for spacetime thumb keys (qmk#18605)
  [Keyboard] Add encoder map support for GMMK Pro (qmk#18600)
  [Keyboard] Add waffling60 Rev D ANSI Hotswap (qmk#18579)
  [Keyboard] Add waffling60 Rev D Solder (qmk#18580)
  Added new issue template for "Other issues" (qmk#18578)
  docs: some typos (qmk#18582)
  RESET → QK_BOOT in nyhxis/nfr_70 (qmk#18607)
  Fixup builds for mechlovin/infinity87. (qmk#18604)
  fix: ps2_interrupt.c failed to compile (qmk#18597)
  Add C++ prerequisite for Fedora. (qmk#18602)
  [Keyboard] Add Nyhxis NFR-70 keyboard (qmk#17562)
  [Docs] Add a note about the print function. (qmk#17737)
  [Keyboard] Twig50 (qmk#18085)
  [Docs] Clarify how to use the bootloader LED for rp2040 (qmk#18585)
  Bump anothrNick/github-tag-action from 1.51.0 to 1.52.0 (qmk#18590)
  define `oled_write_ln_P` as `oled_write_ln` for non-AVR MCUs (qmk#18589)
  ...
quinnyo pushed a commit to quinnyo/qmk_firmware that referenced this pull request Oct 21, 2022
aivalja pushed a commit to aivalja/qmk_firmware that referenced this pull request Nov 7, 2022
ramonimbao pushed a commit to ramonimbao/qmk_firmware that referenced this pull request Nov 28, 2022
nolanseaton pushed a commit to nolanseaton/qmk_firmware that referenced this pull request Jan 23, 2023
@michael-mml
Copy link

What's the difference between the flag set here and this new one under default/?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
keymap via Adds via keymap and/or updates keyboard for via support
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants