starred/psst

Fork 0

mirror of https://github.com/jpochyla/psst.git synced 2026-04-27 23:45:57 +03:00

[GH-ISSUE #13] Add UI for incorrect credentials? #11

New issue

Closed

opened 2026-02-28 14:29:56 +03:00 by kerem · 6 comments

kerem commented

2026-02-28 14:29:56 +03:00

Owner

Originally created by @jasikpark on GitHub (Jan 4, 2021).
Original GitHub issue: https://github.com/jpochyla/psst/issues/13

Currently there is logging for incorrect credentials, but there is no way to see in the interface that the credentials have been saved, updated, are incorrect, or are already saved and won't be replaced since they are the same.

Originally created by @jasikpark on GitHub (Jan 4, 2021). Original GitHub issue: https://github.com/jpochyla/psst/issues/13 Currently there is logging for incorrect credentials, but there is no way to see in the interface that the credentials have been saved, updated, are incorrect, or are already saved and won't be replaced since they are the same. <img width="1792" alt="Screen Shot 2021-01-04 at 5 02 40 PM" src="https://user-images.githubusercontent.com/10626596/103588483-e5b43280-4eae-11eb-841d-74105dae0e26.png">

kerem added the

enhancement

label

2026-02-28 14:29:56 +03:00

kerem referenced this issue

2026-02-28 14:29:56 +03:00

[GH-ISSUE #11] [GUI] Hovered item not correctly highlighted after scrolling down. #12

kerem closed this issue

2026-02-28 14:29:56 +03:00

kerem commented

2026-02-28 14:29:56 +03:00

Author

Owner

@jasikpark commented on GitHub (Jan 4, 2021):

I keep getting "already configured" when trying to update my credentials

@jasikpark commented on GitHub (Jan 4, 2021): I keep getting "already configured" when trying to update my credentials

kerem commented

2026-02-28 14:29:56 +03:00

Author

Owner

@jasikpark commented on GitHub (Jan 4, 2021):

Hmmmm ok so the logging is just simply not very clear, rather than the application being wrong, since I tried to clear my credentials and save and when I checked the config.json in application support, the credentials were cleared.

@jasikpark commented on GitHub (Jan 4, 2021): Hmmmm ok so the logging is just simply not very clear, rather than the application being wrong, since I tried to clear my credentials and save and when I checked the config.json in application support, the credentials were cleared.

kerem commented

2026-02-28 14:29:56 +03:00

Author

Owner

@jpochyla commented on GitHub (Jan 5, 2021):

So I think the strategy here would be to try to connect in the config dialog itself, and show the error there. What is problematic in the current codebase is "restarting" already running session with different credentials, and that's what these "already configured" errors mean. It should work if you restart the app, although I agree it's not ideal.

@jpochyla commented on GitHub (Jan 5, 2021): So I think the strategy here would be to try to connect in the config dialog itself, and show the error there. What is problematic in the current codebase is "restarting" already running session with different credentials, and that's what these "already configured" errors mean. It should work if you restart the app, although I agree it's not ideal.

kerem commented

2026-02-28 14:29:57 +03:00

Author

Owner

@OpenUAS commented on GitHub (Jan 7, 2021):

+1 for this change welcomed. The good thing is, pssst compiles and runs well on Ubuntu 20.04, now only if I would have a Spotify account ;) Tried it since this is good example for a rust app with Druid GUI.

@OpenUAS commented on GitHub (Jan 7, 2021): +1 for this change welcomed. The good thing is, pssst compiles and runs well on Ubuntu 20.04, now only if I would have a Spotify account ;) Tried it since this is good example for a rust app with Druid GUI.

kerem commented

2026-02-28 14:29:57 +03:00

Author

Owner

@brightly-salty commented on GitHub (Mar 7, 2021):

I think it would also be useful to include a string representation of the error. AuthFailed { 11 } is the error I got, but it took digging through keyexchange.proto to find that this meant "Premium account required".

@brightly-salty commented on GitHub (Mar 7, 2021): I think it would also be useful to include a string representation of the error. `AuthFailed { 11 }` is the error I got, but it took digging through keyexchange.proto to find that this meant "Premium account required".

kerem commented

2026-02-28 14:29:57 +03:00

Author

Owner

@jpochyla commented on GitHub (Mar 13, 2021):

Credential dialog is a bit nicer now, closing. Feel free to open issues in case of any problems!

@jpochyla commented on GitHub (Mar 13, 2021): Credential dialog is a bit nicer now, closing. Feel free to open issues in case of any problems!