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

Update props-system-storageproviderstatus.md #1988

Open
wants to merge 1 commit into
base: docs
Choose a base branch
from

Conversation

woojou
Copy link

@woojou woojou commented Feb 26, 2025

The string is wrong, it should be StorageProviderState rather than StorageProviderStatus.
After using the correct string, RetrievePropertiesAsync can return right values. Otherwise, it keeps returning 0 despite of the real status of files.

The string is wrong, it should be StorageProviderState rather than StorageProviderStatus.
Copy link
Contributor

@woojou : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@alvinashcraft
Copy link
Contributor

alvinashcraft commented Mar 4, 2025

Hi @woojou. Both StorageProviderStatus and StorageProviderState exist. I checked the source code, and System.StorageProviderStatus has propID 110, so the existing documentation is correct. System.StorageProviderState has propID of 3.

I will check why we don't have a page for StorageProviderState.

@alvinashcraft alvinashcraft requested review from alvinashcraft and removed request for jwmsft March 4, 2025 18:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants