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

Added PWA support spec #563

Merged
merged 6 commits into from
Oct 6, 2020
Merged

Added PWA support spec #563

merged 6 commits into from
Oct 6, 2020

Conversation

amrutha95
Copy link
Contributor

@amrutha95 amrutha95 commented Sep 11, 2020

Added PWA support spec for community review

Microsoft Reviewers: Open in CodeFlow

@amrutha95 amrutha95 requested a review from a team as a code owner September 11, 2020 20:46

### System requirements

Windows 10 v2004 or newer
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The client is intended to work back to 1709, how will this be validated in the client to ensure users on earlier unsupported versions of Windows 10?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If the YAML manifest indicates the proper minimum OS version, the client will handle the check and enforcement. And no, it is not in the index at this time.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This implementation for now supports all VB+ devices. There is some ongoing work on PWA installs on VB- as well, which can be integrated in the future.


### Reliability

This is not expected to impact reliability. It will be available as an experimental feature so users can disable it any time.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The spec is not for the experimental feature, it should be for the released one. The experimental stage is along the path.

Copy link
Contributor Author

@amrutha95 amrutha95 Sep 14, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay, I removed that sentence on this commit : 5b642f5. Had to rename the file there was a mismatch in the issue number.

@denelon denelon merged commit 306753e into microsoft:master Oct 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants