Add support for OwLab Suit80

[owlab-git]

Description

QMK Pull Request

Checklist

• The VIA support for this keyboard is in QMK master already (MANDATORY)
• The VIA definition follows the guide here: https://caniusevia.com/docs/layouts
• 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

[owlab-git]

bump

[str-dst]

There is nothing to bump here.

The QMK PR for this keyboard is not yet merged into master.
Which you btw failed to link to in this PR (which is very clearly listed as a requirement in the template).
qmk/qmk_firmware#14362

You apparently even failed to properly read what the template says.

The VIA support for this keyboard is in QMK master already (MANDATORY)

This MANDATORY is not there as a joke.
You marked this off as complete even though it isn't

[str-dst]

Updated definitions with colors per VIA's KLE JSON rules, see this gist. Nice board!

The files in that gist aren't correct either...

[owlab-git]

Modified

[str-dst]

ISO layout file is still wrong...

[gknapp]

Thanks for the feedback. It'd help if you could add a short note why, so we don't duplicate effort - you find the mistake, then I try to find the mistake.

[str-dst]

Thanks for the feedback. It'd help if you could add a short note why, so we don't duplicate effort - you find the mistake, then I try to find the mistake.

I'm talking directly with the Owlab guys on Discord (I worked with them before).
That's why I didn't add the fixed jsons here.

There are also plenty of other TKLs in this repo that you can use as a reference how a correct json should look like.

For the ISO version the correct layout would look like this

[owlab-git]

ISO file modified.

[owlab-git]

Already modified, please check

[wilba]

@kb-elmo do I merge or not?

[str-dst]

@kb-elmo do I merge or not?

Yeah looks fine I think 😄