search

microsoft / AzureStorageExplorer

Easily manage the contents of your storage account with Azure Storage Explorer. Upload, download, and manage blobs, files, queues, tables, and Cosmos DB entities. Gain easy access to manage your virtual machine disks. Work with either Azure Resource Manager or classic storage accounts, plus manage and configure cross-origin resource sharing (CORS) rules.
Creative Commons Attribution 4.0 International
372 stars 85 forks source link

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

Closed v-xianya closed 2 months ago

v-xianya commented 3 months ago

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.

craxal commented 3 months ago

@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.

v-xianya commented 3 months ago

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.
craxal commented 3 months ago

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

v-xianya commented 3 months ago

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

craxal commented 2 months ago

Same root cause as #7976.

craxal commented 2 months ago

Fix merged.

v-xianya commented 2 months ago

Verified this issue on the main build 20240724.1. Fixed.