Skip to content
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

The profile items with connection type SAS 9.4(remote) are deleted unexpectedly if trying to cancel the update #389

Closed
Zhirong2022 opened this issue Jul 11, 2023 · 0 comments · Fixed by #392
Assignees
Labels
bug Something isn't working

Comments

@Zhirong2022
Copy link
Collaborator

Describe the bug
Install the SAS extension base on the latest Main branch. Create a new SAS profile with connection type SAS 9.4(remote). Update the newly created profile or update a valid one with connection type SAS 9.4(remote), select the profile and select the connection type as SAS 9.4(remote), it will go to SAS 9 SSH server dialog. Type Esc to cancel all of the updates. The profile items will be deleted unexpectedly if perform Cancel action.

Steps to reproduce
Steps to reproduce the behavior:

  1. Create a new SAS profile with connection type SAS 9.4(remote)
  2. Update the newly created profile or an existed one with connection type SAS 9.4(remote) through 'SAS: Update Connection Profile' on command palette
  3. Select the profile
  4. Select the connection type
  5. Type Esc to cancel the following update

Expected behavior
Some of the profile items are not deleted if cancel the updates

Screenshots
image

Environment (please complete the following information):
Client OS: [e.g. Windows 11]
Extension version: [e.g. v1.0.0]

@Zhirong2022 Zhirong2022 added the bug Something isn't working label Jul 11, 2023
@scnwwu scnwwu assigned scnwwu and unassigned clangsmith and 2TomLi Jul 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants