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 ProtoKeeb keyboard request #23991

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

Conversation

atechofficials
Copy link

Description

The Protokeeb is a versatile 16-key (4 x 4) Numpad/Macropad/MIDI device designed by A-Tech Officials. It is perfect for anyone looking to get started with custom keyboards. This open-source hardware project is based on a Raspberry Pi Pico development board, making it easy to learn, build, and modify. Designed for developers and enthusiasts alike, the Protokeeb features per-key south-facing ARGB LEDs and much more.

For more information, visit the Protokeeb GitHub page.

Types of Changes

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

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

atechofficials and others added 28 commits February 16, 2024 21:05
…nto dev_branch

Merging the Master branch with the dev_branch. All the Protokeeb QMK firmware
development will be carried out in the dev_branch.
…dev_branch

QMK dev_branch updated to the latest qmk firmware
…g to the QMK's new changelogs (renamed Protokeeb rev1 info.json to keyboard.json) and much more...
…hanges according to the PR Checklist documentation. Reorganized existing keymaps and added a new keymap.
@atechofficials
Copy link
Author

Project Update:

  • Successfully implemented the suggested method for the encoder push button functionality.
  • Firmware compiled successfully and also tested the encoder push button and it is working as intended. VIA keymap is also working as expected.

The only problem that I have found with VIA configurator is that there is no method to implement/display the push button key and the encoder as a single entity in the VIA configurator UI. Also, to let the end users access the push button key to assign their keycodes I also modified the Protokeeb Rev1 VIA Layout JSON file which I am going to upload to the VIA GitHub repository by creating a pull request later.

ProtoKeeb Rev1 VIA JSON file

{
    "name": "Protokeeb",
    "vendorId": "0x419A",
    "productId": "0x1717",
    "matrix": {
        "rows": 4,
        "cols": 5
    },
    "keycodes": ["qmk_lighting"],
    "menus": ["qmk_rgblight"],
    "layouts": {
        "keymap": [
            [
                {
                    "y": 0.3,
                    "x": 2,
                    "a": 4
                },
                "0,4",
                {
                    "a": 7
                },
                "e0"
            ],
            [
                {
                    "a": 4
                },
                "0,0",
                "0,1",
                "0,2",
                "0,3"
            ],
            [
                "1,0",
                "1,1",
                "1,2",
                "1,3"
            ],
            [
                "2,0",
                "2,1",
                "2,2",
                "2,3"
            ],
            [
                {
                    "c": "#aaaaaa"
                },
                "3,0",
                {
                    "c": "#cccccc"
                },
                "3,1",
                "3,2",
                "3,3"
            ]
        ]
    }
}

keyboards/protokeeb/rev1/rev1.c Show resolved Hide resolved
keyboards/protokeeb/rules.mk Outdated Show resolved Hide resolved
keyboards/protokeeb/rev1/keyboard.json Outdated Show resolved Hide resolved
keyboards/protokeeb/keymaps/default/rules.mk Outdated Show resolved Hide resolved
keyboards/protokeeb/keymaps/midi/rules.mk Outdated Show resolved Hide resolved
keyboards/protokeeb/keymaps/via/rules.mk Outdated Show resolved Hide resolved
atechofficials and others added 7 commits June 27, 2024 13:59
Co-authored-by: jack <jack@pngu.org>
Co-authored-by: jack <jack@pngu.org>
Co-authored-by: jack <jack@pngu.org>
Co-authored-by: jack <jack@pngu.org>
Co-authored-by: jack <jack@pngu.org>
….json file and also formatted the C code in rev1.c file as requested by QMK Maintainer in Pull Request.
@atechofficials
Copy link
Author

After merging the master branch of my QMK fork with the "dev_branch" to keep it updated with the latest QMK firmware changes, the QMK CI Build check in this pull request is taking significantly longer than my previous commits. It's been 11 hours and the test is still not completed. Can you please look into this issue?

@atechofficials
Copy link
Author

Just a heads up that all the requested changes have been made in the Protokeeb keyboard firmware files. The firmware has compiled successfully on my local machine, and I've tested everything to ensure it works as intended. Please take a moment to review the changes and then proceed with the pull request.

@github-actions github-actions bot added python cli qmk cli command labels Jun 29, 2024
@fauxpark
Copy link
Member

You've merged incorrectly, reverting a bunch of newer changes upstream. Please fix this.

@atechofficials
Copy link
Author

You've merged incorrectly, reverting a bunch of newer changes upstream. Please fix this.

I've merged the master branch of QMK with my QMK fork's dev_branch to address this issue. If there are any additional steps or suggestions regarding this matter, please feel free to let me know.

@atechofficials
Copy link
Author

Dear QMK Maintainers,
@fauxpark @drashna @waffle87

I wanted to follow up on my previous communication regarding the pull request I submitted. Despite having addressed all the requested changes, I have not yet received a response from any of the QMK Maintainers. I understand that you receive numerous pull requests daily and that reviewing and responding to each one promptly may not be feasible. I kindly request your attention to further process my pull request at your earliest convenience.

Thank you for your time and consideration.

@fauxpark
Copy link
Member

Please look at the "Files changed" tab. There are a bunch of irrelevant changes that need to be undone.

@atechofficials
Copy link
Author

Please look at the "Files changed" tab. There are a bunch of irrelevant changes that need to be undone.

My knowledge of git is limited. Please guide me through the steps needed to fix the issue.

@atechofficials
Copy link
Author

Fixed the issues that occurred due to the incorrect merging of branches and content. Reverted back to the commit where everything was working as intended.

@atechofficials
Copy link
Author

Dear QMK Maintainers @waffle87 @drashna

Subject: Review Request for Pull Request

I have completed all the requested changes. The firmware has been successfully compiled and everything is working as intended. I submitted a request for a re-review of my pull request 3 days ago. I kindly request the QMK maintainers to review my pull request as soon as possible.

keyboards/protokeeb/rev1/config.h Outdated Show resolved Hide resolved
keyboards/protokeeb/rev1/config.h Outdated Show resolved Hide resolved
atechofficials and others added 2 commits July 6, 2024 00:26
Co-authored-by: Drashna Jaelre <drashna@live.com>
Co-authored-by: Drashna Jaelre <drashna@live.com>
@atechofficials
Copy link
Author

Hello @drashna and @waffle87,

It's been two weeks since I submitted this pull request. I've made all the requested file modifications and optimizations from the QMK maintainers. I'm not sure why there's a delay in processing this pull request. Could you please either merge it or let me know if my keyboard contribution isn't considered worthy for the QMK project?

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
keyboard 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