Update skiller_sgk50_s4 keyboard

[itarze]

Description

Update skiller_sgk50_s4 keyboard.

QMK Pull Request

qmk/qmk_firmware#24784

VIA Keymap Pull Request

Checklist

• The VIA support for this keyboard is MERGED in QMK master already (MANDATORY)
• VIA keymap is MERGED in VIA userspace 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 formatted the JSON file to have consistent formatting with the rest of the repository.
• 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]

The context for the update?

The changes proposed would not work at all with the board definitions in QMK code here and by extension in the VIA keymap as well in here.

Closing this PR for the moment, feel free to reopen it again once you apply the appropriate changes in QMK main as well as VIA userspace

[itarze]

@Cipulot The changed keymap has been merged into QMK. This PR

[Cipulot]

@itarze thanks for pointing this out, this PR will then be put on hold since the QMK main PR targets develop, it being a Keeb change.
I'd also suggest to open a PR for the VIA keymap over in teh VIA userspace that will be merged once the next breaking changes are merged.

reason why the changes need to be n QMK main, and then in the userspace, is because the build script that gives out binaries in the via website targets main branch.

[itarze]

the-via/qmk_userspace_via#64