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.

Sign up for GitHub

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

Jump to bottom

The error details are unclear when creating a disk snapshot with whitespaces #4262

Closed
v-xianya opened this issue Mar 25, 2021 · 2 comments
Closed

The error details are unclear when creating a disk snapshot with whitespaces #4262

v-xianya opened this issue Mar 25, 2021 · 2 comments
Assignees
@MRayermannMSFT
Labels
⚙️ disks Related to managed disks 🧪 testing Found through regular testing
Milestone
1.20.0

Comments

@v-xianya
Copy link
Member

Storage Explorer Version: 1.19.0-dev
Build Number: 20210325.3
Branch: main
Platform/OS: Windows 10/ Linux Ubuntu 18.04/ MacOS Catalina
Architecture: ia32/x64
Regression From: Not a regression

Steps to Reproduce

  1. Expand the 'Disks' node -> Open one resource group.
  2. Upload one disk to it -> Click 'Create Snapshot'.
  3. Delete the default snapshot name then type whitespaces as the snapshot name -> Click 'Create'.
  4. Check whether the error details are clear.

Expected Experience

The error details are clear.

Actual Experience

The error details are unclear.
image

Additional Context

  1. Show the below behavior when trying to type whitespaces as the disk/ disk snapshot name.
    image

  2. This issue also reproduces when uploading one disk then specify whitespaces as the disk name.

image
@v-xianya v-xianya added 🧪 testing Found through regular testing ⚙️ disks Related to managed disks labels Mar 25, 2021
@v-xianya v-xianya changed the title The error details are unclear when creating disk snapshot with whitespaces The error details are unclear when creating a disk snapshot with whitespaces Mar 25, 2021
@MRayermannMSFT MRayermannMSFT added this to the 1.20.0 milestone Mar 29, 2021
@MRayermannMSFT MRayermannMSFT self-assigned this Mar 29, 2021
@MRayermannMSFT
Copy link
Member

MRayermannMSFT commented May 3, 2021 •
edited
Loading

Fix merged into main. Feel free to turn this into a recurring test case (P2?).

@v-xianya
Copy link
Member Author

v-xianya commented May 6, 2021

@MRayermannMSFT
I added this to the regular test case and marked it as P2.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@MRayermannMSFT MRayermannMSFT

Labels
⚙️ disks Related to managed disks 🧪 testing Found through regular testing
Projects
None yet
Milestone
1.20.0
Development

No branches or pull requests
2 participants
@MRayermannMSFT @v-xianya