AzureStorageExplorer icon indicating copy to clipboard operation
AzureStorageExplorer copied to clipboard

There is no refresh notification when deleting a blob container with the setting 'Auto Refresh On Change' unchecked

Open v-xianya opened this issue 1 year ago • 4 comments

Storage Explorer Version: 1.35.0-dev (98) Build Number: 20240607.1 Branch: main Platform/OS: Windows11/Windows 10/Linux Ubuntu 22.04/MacOS Sonoma 14.5(Apple M1 Pro) Architecture: arm64/x64/x64/arm64 How Found: From running test cases Regression From: Previous release (1.34.0)

Steps to Reproduce

  1. Open ‘Settings -> Data Explorers' -> Disable the setting 'Auto Refresh On Change'.
  2. Expand one storage account -> Right click the 'Blob Containers' node.
  3. Create a blob container -> Right click the 'Blob Containers' node -> Click 'View All'.
  4. Select the blob container on the view all containers explorer -> Delete it.
  5. Check whether there is a refresh notification.

Expected Experience

There is a refresh notification. image

Actual Experience

There is no refresh notification.

Additional Context

This issue can't be reproduced consistently.

v-xianya avatar Jun 07 '24 05:06 v-xianya

@v-xianya I'm not able to reproduce. Is it possible that the underlying issue with #7983 is also contributing to this? If you can figure out how to consistently reproduce this, that would be helpful.

craxal avatar Jun 12 '24 18:06 craxal

Hi @craxal could you run the steps below to see if this issue can be reproduced?

  1. Disable the setting 'Auto Refresh On Change'.
  2. Expand one storage account -> Blob Containers.
  3. Open view all containers tab.
  4. Then create a blob container (either the tree node or the View All Containers tab) -> Go to view all containers tab.
  5. Click 'Yes' on the refresh notification -> Then delete it from view all containers explorer.

v-xianya avatar Jun 13 '24 06:06 v-xianya

I have followed those exact steps multiple times and have not been able to reproduce. Do you have app logs to share (debug verbosity)?

craxal avatar Jun 13 '24 17:06 craxal

Here is the log: 2024-06-14_023308.zip

v-xianya avatar Jun 14 '24 02:06 v-xianya

Same root cause as #7976.

craxal avatar Jul 17 '24 20:07 craxal

Fix merged.

craxal avatar Jul 23 '24 22:07 craxal

Verified this issue on the main build 20240724.1. Fixed.

v-xianya avatar Jul 24 '24 02:07 v-xianya