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 support for plaque AD #2209

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

MaiTheSan
Copy link
Contributor

@MaiTheSan MaiTheSan commented Mar 26, 2024

Description

Add support for Plaque AD TKL PCB

QMK Pull Request

qmk/qmk_firmware#23359

Checklist

  • The VIA support for this keyboard is MERGED in QMK master already (MANDATORY)
  • The VIA definition follows the guide here: https://caniusevia.com/docs/layouts
  • I have a V3 JSON version for this keyboard definition.(MANDATORY)
  • I have tested this keyboard definition using VIA's "Design" tab.
  • I have tested this keyboard definition with firmware on a device.
  • I have assigned alpha keys and modifier keys with the correct colors.
  • The Vendor ID is not 0xFEED

@Cipulot Cipulot added the pending QMK merge Waiting for merge into QMK master label Mar 27, 2024
@Cipulot
Copy link
Collaborator

Cipulot commented Mar 27, 2024

Adding Pending QMK merge label.

@dunk2k
Copy link
Contributor

dunk2k commented Mar 28, 2024

Alpha keys and modifier keys don't have colours assigned 🙈

"keycodes": ["qmk_lighting"],
"menus": ["qmk_rgb_matrix"],
"layouts": {
"labels": [
Copy link
Contributor

Choose a reason for hiding this comment

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

missing f12 function row (layout)

v3/asmodus/plaquead/plaquead.json Show resolved Hide resolved
@Cipulot
Copy link
Collaborator

Cipulot commented Mar 28, 2024

@dunk2k no point in submitting reviews at this time if the board isn't even merged in QMK main as per requirements.
PRs that don't even meet the PR checklist checks shouldn't even be opened or be marked as "ready for review" if they are not in the stage that allows them to mark all the checks requirements imo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pending QMK merge Waiting for merge into QMK master
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants