the-via / app

GNU General Public License v3.0
923 stars 210 forks source link

Cannot authorise Keychron K1 Pro #199

Open thany opened 1 year ago

thany commented 1 year ago

Whether I connect it through USB or Bluetooth, my Keychron K1 Pro doesn't want to connect. When I try, just nothing happens. No errors in the console, no messages of any kind anywhere whatsoever. At first, I thought it was succesful, it's that silent about it not working.

I've tried it on Chrome 117, on Windows 10.

Only the key tester works, which is not really very useful, with all due respect, because it has no idea which layout my keyboard is. It's TKL, but it has no knowledge of that fact, and I can't tell it what it is either.

Anyway, may I suggest that, when a keyboard cannot authorise for whatever reason, that it displays a message explaining what that's the case? It might help both the user and you (devs) to diagnose what is possibly going wrong. Now I have no way to give you any more useful information, which is a bit of a shame.

DRBragg commented 1 year ago

I am also having a problem connecting to my new Lemokey L3 to get the keys mapped to Mac. Not sure if it will help but this is what I see in the console Screenshot 2023-10-10 at 09 26 07

DRBragg commented 1 year ago

Disregard my above screen shot. It seems the errors were caused by an extension. Still having trouble connecting though...

thany commented 1 year ago

I've done the firmware flash ritual and it still won't work.

I've then done the custom json mapping thing, which I shouldn't have to do as a user (not my words, the app says so), and then going back to the first tab bring me back to square one. I can autorise and authorise to the end of time and it will never work.

Here's an idea: when autorising can't work for whatever reason, display a damn message about it, or something? Something, anything, that will help diagnosing an issue like this better than stumbling around in the dark, blindly trying things that might solve it. Seriously, I can't stand silent fails, they are useless to both users and developers.

thany commented 1 year ago

Got it working with this JSON file - k1_pro_us_rgb_v1.10.json

This one should work on the US ANSI RGB version.

Can this please be included in the app for everyone? I had to back-and-forth with Keychron quite a lot and for this to be the workaround feels pretty lame, tbh 😑

ristomatti commented 1 year ago

With Keychron K3 Pro RGB ISO Nordic I had the same issue on Chrome/Chromium (Linux). However the Electron app version works when combined with the keymap JSON downloaded from https://cdn.shopify.com/s/files/1/0059/0630/1017/files/k3_pro_iso_rgb_v1.01_20230526.json.zip?v=1685152590. I can't remember where I found the link though. :D

thany commented 1 year ago

So it's basically the same deal for the Nordic. I feel there should be a facility for regular users to submit a keymap json.

It seems it's a bit too high of a threshold for most people, who aren't developers, to submit a PR. And even if someone creates a PR, how long until it gets accepted...

kdashg commented 12 months ago

It looks like a PR for the K1 Pro exists, but is stuck: https://github.com/the-via/keyboards/pull/1775

But it does contain the json files that will eventually be included: https://github.com/the-via/keyboards/pull/1775/files