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 keyboard Projectd/65/Projectd_65_ iso #21941

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

Conversation

gskygithub
Copy link
Contributor

Description

ADD project_65_iso keyboard

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

@drashna drashna requested a review from a team January 13, 2024 06:50
Copy link

Thank you for your contribution!
This pull request has been automatically marked as stale because it has not had activity in the last 45 days. It will be closed in 30 days if no further activity occurs. Please feel free to give a status update now, or re-open when it's ready.
For maintainers: Please label with bug, awaiting review, breaking_change, in progress, or on hold to prevent the issue from being re-flagged.

@github-actions github-actions bot added the stale Issues or pull requests that have become inactive without resolution. label Feb 28, 2024
@gskygithub
Copy link
Contributor Author

@fauxpark , could you please take a look at this PR, thanks.

Comment on lines 30 to 31
#define AW20216S_EN_PIN_1 C13
#define AW20216S_EN_PIN_2 C13
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
#define AW20216S_EN_PIN_1 C13
#define AW20216S_EN_PIN_2 C13
#define AW20216S_EN_PIN C13

Comment on lines 122 to 124
setPinInput(SPI_SCK_PIN);
setPinInput(SPI_MOSI_PIN);
setPinInput(SPI_MISO_PIN);
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
setPinInput(SPI_SCK_PIN);
setPinInput(SPI_MOSI_PIN);
setPinInput(SPI_MISO_PIN);
gpio_set_pin_input(SPI_SCK_PIN);
gpio_set_pin_input(SPI_MOSI_PIN);
gpio_set_pin_input(SPI_MISO_PIN);

Copy link
Member

Choose a reason for hiding this comment

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

Please run qmk format-json -i on this file.

Copy link
Member

Choose a reason for hiding this comment

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

This is still outstanding.

Copy link
Member

Choose a reason for hiding this comment

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

Still outstanding, and this file can also be renamed to keyboard.json.

* | | | | */
//{0, CS1_SW4, CS2_SW4, CS3_SW4}, // 0, Esc

{0, CS1_SW1, CS2_SW1, CS3_SW1}, // 0, Esc
Copy link
Member

Choose a reason for hiding this comment

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

These should be swapped around (SW1_CS1 etc).

@github-actions github-actions bot removed the stale Issues or pull requests that have become inactive without resolution. label Mar 1, 2024
@gskygithub gskygithub requested a review from fauxpark March 1, 2024 01:59
Copy link
Member

Choose a reason for hiding this comment

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

This file should be removed. If these definitions are needed, they can be moved to the keyboard level info.json file.

@@ -0,0 +1,25 @@
# ProjectD 65% ISO
![ProjectD 65 ISO](https://imgur.com/2LCEGnJh.png)
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
![ProjectD 65 ISO](https://imgur.com/2LCEGnJh.png)
![ProjectD 65 ISO](https://imgur.com/2LCEGnJ.png)

@gskygithub gskygithub requested a review from waffle87 April 3, 2024 08:13
@drashna drashna requested a review from a team May 2, 2024 22:37
Copy link

Thank you for your contribution!
This pull request has been automatically marked as stale because it has not had activity in the last 45 days. It will be closed in 30 days if no further activity occurs. Please feel free to give a status update now, or re-open when it's ready.
For maintainers: Please label with bug, awaiting review, breaking_change, in progress, or on hold to prevent the issue from being re-flagged.

@github-actions github-actions bot added the stale Issues or pull requests that have become inactive without resolution. label Jun 17, 2024
Copy link
Member

Choose a reason for hiding this comment

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

This file can be deleted.

Copy link
Member

Choose a reason for hiding this comment

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

Still outstanding, and this file can also be renamed to keyboard.json.

Comment on lines 112 to 152
{ "flags": 4, "matrix": [8, 4], "x": 137.5, "y": 10 },
{ "flags": 4, "matrix": [7, 3], "x": 150, "y": 10 },
{ "flags": 1, "matrix": [2, 1], "x": 0, "y": 20 },
{ "flags": 4, "matrix": [1, 2], "x": 17.5, "y": 20 },
{ "flags": 4, "matrix": [2, 2], "x": 27.5, "y": 20 },
{ "flags": 4, "matrix": [3, 2], "x": 37.5, "y": 20 },
{ "flags": 4, "matrix": [4, 2], "x": 47.5, "y": 20 },
{ "flags": 4, "matrix": [4, 3], "x": 57.5, "y": 20 },
{ "flags": 4, "matrix": [5, 3], "x": 67.5, "y": 20 },
{ "flags": 4, "matrix": [5, 2], "x": 77.5, "y": 20 },
{ "flags": 4, "matrix": [6, 2], "x": 87.5, "y": 20 },
{ "flags": 4, "matrix": [7, 2], "x": 97.5, "y": 20 },
{ "flags": 4, "matrix": [8, 2], "x": 107.5, "y": 20 },
{ "flags": 4, "matrix": [8, 3], "x": 117.5, "y": 20 },
{ "flags": 4, "matrix": [7, 5], "x": 127.5, "y": 20 },
{ "flags": 4, "matrix": [2, 6], "x": 150, "y": 20 },
{ "flags": 4, "matrix": [0, 0], "x": 0, "y": 30 },
{ "flags": 4, "matrix": [0, 4], "x": 12.5, "y": 30 },
{ "flags": 4, "matrix": [1, 4], "x": 22.5, "y": 30 },
{ "flags": 4, "matrix": [2, 4], "x": 32.5, "y": 30 },
{ "flags": 4, "matrix": [3, 4], "x": 42.5, "y": 30 },
{ "flags": 4, "matrix": [4, 4], "x": 52.5, "y": 30 },
{ "flags": 4, "matrix": [4, 5], "x": 62.5, "y": 30 },
{ "flags": 4, "matrix": [5, 5], "x": 72.5, "y": 30 },
{ "flags": 4, "matrix": [5, 4], "x": 82.5, "y": 30 },
{ "flags": 4, "matrix": [6, 4], "x": 92.5, "y": 30 },
{ "flags": 4, "matrix": [7, 4], "x": 102.5, "y": 30 },
{ "flags": 4, "matrix": [8, 5], "x": 112.5, "y": 30 },
{ "flags": 4, "matrix": [0, 7], "x": 122.5, "y": 30 },
{ "flags": 4, "matrix": [1, 6], "x": 140, "y": 30 },
{ "flags": 4, "matrix": [6, 3], "x": 150, "y": 30 },
{ "flags": 4, "matrix": [0, 6], "x": 0, "y": 40 },
{ "flags": 4, "matrix": [0, 5], "x": 12.5, "y": 40 },
{ "flags": 4, "matrix": [0, 2], "x": 25, "y": 40 },
{ "flags": 4, "x": 61.5, "y": 40 },
{ "flags": 4, "x": 62.5, "y": 40 },
{ "flags": 4, "matrix": [0, 1], "x": 65, "y": 40 },
{ "flags": 4, "x": 67.5, "y": 40 },
{ "flags": 4, "x": 68.5, "y": 40 },
{ "flags": 4, "matrix": [3, 6], "x": 100, "y": 40 },
{ "flags": 4, "matrix": [3, 3], "x": 112.5, "y": 40 },
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
{ "flags": 4, "matrix": [8, 4], "x": 137.5, "y": 10 },
{ "flags": 4, "matrix": [7, 3], "x": 150, "y": 10 },
{ "flags": 1, "matrix": [2, 1], "x": 0, "y": 20 },
{ "flags": 4, "matrix": [1, 2], "x": 17.5, "y": 20 },
{ "flags": 4, "matrix": [2, 2], "x": 27.5, "y": 20 },
{ "flags": 4, "matrix": [3, 2], "x": 37.5, "y": 20 },
{ "flags": 4, "matrix": [4, 2], "x": 47.5, "y": 20 },
{ "flags": 4, "matrix": [4, 3], "x": 57.5, "y": 20 },
{ "flags": 4, "matrix": [5, 3], "x": 67.5, "y": 20 },
{ "flags": 4, "matrix": [5, 2], "x": 77.5, "y": 20 },
{ "flags": 4, "matrix": [6, 2], "x": 87.5, "y": 20 },
{ "flags": 4, "matrix": [7, 2], "x": 97.5, "y": 20 },
{ "flags": 4, "matrix": [8, 2], "x": 107.5, "y": 20 },
{ "flags": 4, "matrix": [8, 3], "x": 117.5, "y": 20 },
{ "flags": 4, "matrix": [7, 5], "x": 127.5, "y": 20 },
{ "flags": 4, "matrix": [2, 6], "x": 150, "y": 20 },
{ "flags": 4, "matrix": [0, 0], "x": 0, "y": 30 },
{ "flags": 4, "matrix": [0, 4], "x": 12.5, "y": 30 },
{ "flags": 4, "matrix": [1, 4], "x": 22.5, "y": 30 },
{ "flags": 4, "matrix": [2, 4], "x": 32.5, "y": 30 },
{ "flags": 4, "matrix": [3, 4], "x": 42.5, "y": 30 },
{ "flags": 4, "matrix": [4, 4], "x": 52.5, "y": 30 },
{ "flags": 4, "matrix": [4, 5], "x": 62.5, "y": 30 },
{ "flags": 4, "matrix": [5, 5], "x": 72.5, "y": 30 },
{ "flags": 4, "matrix": [5, 4], "x": 82.5, "y": 30 },
{ "flags": 4, "matrix": [6, 4], "x": 92.5, "y": 30 },
{ "flags": 4, "matrix": [7, 4], "x": 102.5, "y": 30 },
{ "flags": 4, "matrix": [8, 5], "x": 112.5, "y": 30 },
{ "flags": 4, "matrix": [0, 7], "x": 122.5, "y": 30 },
{ "flags": 4, "matrix": [1, 6], "x": 140, "y": 30 },
{ "flags": 4, "matrix": [6, 3], "x": 150, "y": 30 },
{ "flags": 4, "matrix": [0, 6], "x": 0, "y": 40 },
{ "flags": 4, "matrix": [0, 5], "x": 12.5, "y": 40 },
{ "flags": 4, "matrix": [0, 2], "x": 25, "y": 40 },
{ "flags": 4, "x": 61.5, "y": 40 },
{ "flags": 4, "x": 62.5, "y": 40 },
{ "flags": 4, "matrix": [0, 1], "x": 65, "y": 40 },
{ "flags": 4, "x": 67.5, "y": 40 },
{ "flags": 4, "x": 68.5, "y": 40 },
{ "flags": 4, "matrix": [3, 6], "x": 100, "y": 40 },
{ "flags": 4, "matrix": [3, 3], "x": 112.5, "y": 40 },
{ "flags": 4, "matrix": [8, 4], "x": 137, "y": 10 },
{ "flags": 4, "matrix": [7, 3], "x": 150, "y": 10 },
{ "flags": 1, "matrix": [2, 1], "x": 0, "y": 20 },
{ "flags": 4, "matrix": [1, 2], "x": 17, "y": 20 },
{ "flags": 4, "matrix": [2, 2], "x": 27, "y": 20 },
{ "flags": 4, "matrix": [3, 2], "x": 37, "y": 20 },
{ "flags": 4, "matrix": [4, 2], "x": 47, "y": 20 },
{ "flags": 4, "matrix": [4, 3], "x": 57, "y": 20 },
{ "flags": 4, "matrix": [5, 3], "x": 67, "y": 20 },
{ "flags": 4, "matrix": [5, 2], "x": 77, "y": 20 },
{ "flags": 4, "matrix": [6, 2], "x": 87, "y": 20 },
{ "flags": 4, "matrix": [7, 2], "x": 97, "y": 20 },
{ "flags": 4, "matrix": [8, 2], "x": 107, "y": 20 },
{ "flags": 4, "matrix": [8, 3], "x": 117, "y": 20 },
{ "flags": 4, "matrix": [7, 5], "x": 127, "y": 20 },
{ "flags": 4, "matrix": [2, 6], "x": 150, "y": 20 },
{ "flags": 4, "matrix": [0, 0], "x": 0, "y": 30 },
{ "flags": 4, "matrix": [0, 4], "x": 12, "y": 30 },
{ "flags": 4, "matrix": [1, 4], "x": 22, "y": 30 },
{ "flags": 4, "matrix": [2, 4], "x": 32, "y": 30 },
{ "flags": 4, "matrix": [3, 4], "x": 42, "y": 30 },
{ "flags": 4, "matrix": [4, 4], "x": 52, "y": 30 },
{ "flags": 4, "matrix": [4, 5], "x": 62, "y": 30 },
{ "flags": 4, "matrix": [5, 5], "x": 72, "y": 30 },
{ "flags": 4, "matrix": [5, 4], "x": 82, "y": 30 },
{ "flags": 4, "matrix": [6, 4], "x": 92, "y": 30 },
{ "flags": 4, "matrix": [7, 4], "x": 102, "y": 30 },
{ "flags": 4, "matrix": [8, 5], "x": 112, "y": 30 },
{ "flags": 4, "matrix": [0, 7], "x": 122, "y": 30 },
{ "flags": 4, "matrix": [1, 6], "x": 140, "y": 30 },
{ "flags": 4, "matrix": [6, 3], "x": 150, "y": 30 },
{ "flags": 4, "matrix": [0, 6], "x": 0, "y": 40 },
{ "flags": 4, "matrix": [0, 5], "x": 12, "y": 40 },
{ "flags": 4, "matrix": [0, 2], "x": 25, "y": 40 },
{ "flags": 4, "x": 61, "y": 40 },
{ "flags": 4, "x": 62, "y": 40 },
{ "flags": 4, "matrix": [0, 1], "x": 65, "y": 40 },
{ "flags": 4, "x": 67, "y": 40 },
{ "flags": 4, "x": 68, "y": 40 },
{ "flags": 4, "matrix": [3, 6], "x": 100, "y": 40 },
{ "flags": 4, "matrix": [3, 3], "x": 112, "y": 40 },

@github-actions github-actions bot removed the stale Issues or pull requests that have become inactive without resolution. label Jun 18, 2024
@gskygithub gskygithub requested a review from fauxpark June 19, 2024 01:57
Co-authored-by: Duncan Sutherland <dunk2k_2000@hotmail.com>
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

5 participants