[PR #342] [MERGED] Don't close initial login window if authentication failed #489

New issue

Closed

opened 2026-02-28 14:33:11 +03:00 by kerem · 0 comments

kerem commented

2026-02-28 14:33:11 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/jpochyla/psst/pull/342
Author: @Insprill
Created: 10/8/2022
Status: ✅ Merged
Merged: 10/12/2022
Merged by: @jpochyla

Base: master ← Head: fix/login-window

📝 Commits (1)

2c3ea00 Don't close the login window if authentication failed

📊 Changes

1 file changed (+15 additions, -11 deletions)

View changed files

📝 psst-gui/src/ui/preferences.rs (+15 -11)

📄 Description

When initially logging in, even if authentication failed, the login window will be closed and the main window will be opened without a clear indication of why it's blank. This simply makes it so that window is only closed if authentication was successful.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/jpochyla/psst/pull/342 **Author:** [@Insprill](https://github.com/Insprill) **Created:** 10/8/2022 **Status:** ✅ Merged **Merged:** 10/12/2022 **Merged by:** [@jpochyla](https://github.com/jpochyla) **Base:** `master` ← **Head:** `fix/login-window` --- ### 📝 Commits (1) - [`2c3ea00`](https://github.com/jpochyla/psst/commit/2c3ea00328904201c8a773a6f63f0d8c9372d15e) Don't close the login window if authentication failed ### 📊 Changes **1 file changed** (+15 additions, -11 deletions) <details> <summary>View changed files</summary> 📝 `psst-gui/src/ui/preferences.rs` (+15 -11) </details> ### 📄 Description When initially logging in, even if authentication failed, the login window will be closed and the main window will be opened without a clear indication of why it's blank. This simply makes it so that window is only closed if authentication was successful. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>