Add default instance auto-connect on startup#826
Draft
Faerbit wants to merge 1 commit intoDefGuard:devfrom
Draft
Add default instance auto-connect on startup#826Faerbit wants to merge 1 commit intoDefGuard:devfrom
Faerbit wants to merge 1 commit intoDefGuard:devfrom
Conversation
0c1dff1 to
c7d4286
Compare
Adds a settings dropdown to configure which instance should be automatically connected when the client launches. A companion checkbox enables triggering MFA prompts during auto-connect for locations that require it (disabled by default). Selecting "None" (the default) preserves the existing behaviour of no auto-connect. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
c7d4286 to
14b426a
Compare
Author
|
I would like to agree to your CA, but cannot read it as the link 404s. |
Contributor
|
Thank you @Faerbit for your contribution 🫡 We are in the middle of 2.0 release and have much workflow - as we would like to release it this week. Well get back to review this PR after the release! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📝 New contributors
⚠︎ If the checkbox will not be confirmed - we can't include your contribution in our codebase.
📖 Description
Add default instance auto-connect on startup
Adds a settings dropdown to configure which instance should be automatically connected when the client launches. A companion checkbox enables triggering MFA prompts during auto-connect for locations that require it (disabled by default). Selecting "None" (the default) preserves the existing behaviour of no auto-connect.
Relates to #606. Doesn't necessarily close it, as it only connects to one instance.
🛠️ Dev Branch Merge Checklist:
Documentation
Docs: #62
Testing
I was able to successfully test my changes on the main branch. I rebased the changes to the dev branch, because I assume this is the preferred development workflow. I'm unable to verify my changes on the dev branch, because it currently doesn't build.
🏚️ Main Branch Merge Checklist:
Testing
Documentation