Skip to content

Add command to sign out of multiple accounts at once #288

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Jul 28, 2025

Conversation

isc-bsaviano
Copy link
Collaborator

This PR fixes #282. @gjsjohnmurray I know you had this assigned to you but I wanted to include it in the next Server Manager release along with the bug fix.

gjsjohnmurray
gjsjohnmurray previously approved these changes Jul 28, 2025
Copy link
Collaborator

@gjsjohnmurray gjsjohnmurray left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Neat! I did find that some of my logins couldn't be signed out this way, probably ones that date back to an older version of Server Manager. I am able to sign them out from VS Code's Accounts menu.

When I tried to sign out a recently signed in account together with one of these old ones I noticed that the newer account remained signed in and I wasn't notified of a problem signing it out.

@gjsjohnmurray
Copy link
Collaborator

How about adding the command here?

image

@isc-bsaviano
Copy link
Collaborator Author

Sure, I'll add the command there. I'm not sure why the old sessions wouldn't get signed out. I think I replicated the exact behavior of the removeSession() method.

@isc-bsaviano
Copy link
Collaborator Author

Screenshot 2025-07-28 at 9 10 53 AM

@isc-bsaviano isc-bsaviano merged commit 3aed3ad into intersystems-community:master Jul 28, 2025
3 checks passed
@isc-bsaviano isc-bsaviano deleted the fix-282 branch July 28, 2025 14:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add a way to cull logins more easily
2 participants