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

Gen2 Move Path Does Not Work For Non-Root Paths That Start With / #4465

Closed
3 tasks done
MRayermannMSFT opened this issue May 17, 2021 · 3 comments
Closed
3 tasks done
Assignees
Labels
🪲 bug Issue is not intended behavior 🪲 regression Issue was working in a previous version ⚙️ adls gen2 Related to hierarchical namespaces (ADLS Gen 2) ✅ merged A fix for this issue has been merged
Milestone

Comments

@MRayermannMSFT
Copy link
Member

Preflight Checklist

Storage Explorer Version

1.19.0

Platform

Windows

OS Version

No response

Architecture

x64

Build Number

No response

Regression From

1.17.0

Bug Description

Move a file or folder in gen2 to a non-root directory, start the path in the dialog box with a slash.

Steps to Reproduce

See description.

Actual Experience

It doesn't work.

Expected Experience

It should work.

Additional Context

No response

@MRayermannMSFT MRayermannMSFT added 🪲 bug Issue is not intended behavior 🪲 regression Issue was working in a previous version ⚙️ adls gen2 Related to hierarchical namespaces (ADLS Gen 2) labels May 17, 2021
@MRayermannMSFT MRayermannMSFT self-assigned this May 17, 2021
@MRayermannMSFT MRayermannMSFT added this to the 1.20.0 milestone May 17, 2021
@MRayermannMSFT
Copy link
Member Author

MRayermannMSFT commented May 18, 2021

@v-xianya @v-limu while I'm working on a fix for this, can y'all add a (p1 or p2) test case for this? Basically, for Gen2 move, it should not matter if what you input starts/ends with a / or not.

For example, if I'm moving .DS_Store into test:
image
All of these inputs should work:

  • untitled folder/test
  • /untitled folder/test
  • untitled folder/test/
  • /untitled folder/test/

@v-xianya
Copy link
Member

@MRayermannMSFT
Got it. thank you for the reminder. we added this to regular test cases and marked as P1.

@v-xianya
Copy link
Member

Verified this issue on the latest build 20210521.3. Fixed.

@MRayermannMSFT MRayermannMSFT added the ✅ merged A fix for this issue has been merged label May 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🪲 bug Issue is not intended behavior 🪲 regression Issue was working in a previous version ⚙️ adls gen2 Related to hierarchical namespaces (ADLS Gen 2) ✅ merged A fix for this issue has been merged
Projects
None yet
Development

No branches or pull requests

2 participants