-
Notifications
You must be signed in to change notification settings - Fork 144
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
Change to scope file tree to the known path if root folders dont return correctly #816
Conversation
Can you add a screenshot? |
src/Microsoft.SqlTools.ServiceLayer/FileBrowser/FileBrowserOperation.cs
Outdated
Show resolved
Hide resolved
@udeeshagautam do you know what happened with the code coverage? perhaps some tests are failing? |
@kburtram yes, the last run had -1% decline.. so I re-ran and it went -4.5%. Re-ran and it came back to -0.3% decrease. So continuing with merge... |
@shueybubbles @chlafreniere would you like to take another look or I should close? |
Aims to fix microsoft/azuredatastudio#4767 :
Issue : File browser for back up result does not load when the root folders return are not valid and hence the expand path i.e. the default backup path can not be found in those.
Fix: workaround to scope the file browser tree to the default backup path
Following is how it looks when root not found. (Please note that if there is nothing present in the default folder it will show as empty)