-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Code Quality: Removed Vanara from IStorageNetworkService #16234
base: main
Are you sure you want to change the base?
Code Quality: Removed Vanara from IStorageNetworkService #16234
Conversation
cc0c966
to
c7ac2d3
Compare
b193542
to
c420f19
Compare
There's no memory leak here as far as I can tell. |
I will test it when I have time. But if you make a local folder shared, you will be able to access that folder by UNC path, so you can also test without a network PC. |
Somehow the share menu is grayed out and I can't reenable even tho I messed around Group Policy and some settings. |
@0x5bfa have you looked into this issue further? |
I cant test it anyways |
|
Like mentioned, the option (Share) is grayed out somehow and I tried to make it clickable by looking into Group Policy and some other Windows Settings, and didn't work. Either way, it's fine as is if I should test (idk how to). |
I have found a few issues.
|
Resolved / Related Issues
Steps used to test these changes