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

[Keyboard] IDOBAO ID80v1 folder rename #17265

Merged
merged 6 commits into from
Aug 13, 2022
Merged

Conversation

silvinor
Copy link
Contributor

@silvinor silvinor commented May 31, 2022

Description

IDOBAO support has received numerous support calls with users confused with the "v1" name used in the folder structure introduced in #15992.

The code in the v1 folder is a folder shift from the base of the id80folder and actually pertains to the "v2" boards. Both "v1" (with no per-key under-light, or rather; self-solder per-key) and "v2" (with manufactured white per-key leds) are compatible with that code base - however users seeking code for their "v2" boards are using the "v3" codebase, and this is not compatible as it uses RGB LED's and the RGB Matrix codebase.

This PR seeks to rename the "v1" folder to "v2" (to eliminate confusion), and then add a readme.md and rules.mk file to the "v1" (ensuring backward compatibility).

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

  • N/A (more about code clean-up)

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

@github-actions github-actions bot added keyboard keymap via Adds via keymap and/or updates keyboard for via support labels May 31, 2022
keyboards/idobao/id80/v1/readme.md Outdated Show resolved Hide resolved
keyboards/idobao/id80/v2/readme.md Outdated Show resolved Hide resolved
@drashna drashna requested a review from a team May 31, 2022 21:08
@silvinor silvinor requested a review from drashna May 31, 2022 21:59
@drashna drashna requested a review from a team June 2, 2022 16:17
keyboards/idobao/id80/v2/rules.mk Outdated Show resolved Hide resolved
@silvinor silvinor requested review from fauxpark and drashna June 3, 2022 08:55
@drashna
Copy link
Member

drashna commented Jun 10, 2022

Have these been checked to make sure they still compile?

@silvinor
Copy link
Contributor Author

absolument

image

@drashna drashna requested a review from a team June 11, 2022 00:34
@tzarc tzarc merged commit 52dc16b into qmk:develop Aug 13, 2022
@silvinor silvinor deleted the vr-id80-v2-dev-touch branch August 14, 2022 03:02
nolanseaton pushed a commit to nolanseaton/qmk_firmware that referenced this pull request Jan 23, 2023
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.

4 participants