MicrosoftDocs / edge-developer

Developer documentation for Edge.
https://learn.microsoft.com/microsoft-edge/developer/
Creative Commons Attribution 4.0 International
381 stars 505 forks source link

Add articles "Prerelease testing" & "Self-hosting", use new API #3053

Closed mikehoffms closed 4 months ago

mikehoffms commented 6 months ago

Rendered articles for review:

AB#48929283

learn-build-service-prod[bot] commented 4 months ago

Learn Build status updates of commit 95b1956:

:white_check_mark: Validation status: passed

File Status Preview URL Details
microsoft-edge/toc.yml :white_check_mark:Succeeded View
microsoft-edge/webview2/concepts/distribution.md :white_check_mark:Succeeded View
microsoft-edge/webview2/concepts/versioning.md :white_check_mark:Succeeded View
microsoft-edge/webview2/how-to/prerelease-testing.md :white_check_mark:Succeeded View
microsoft-edge/webview2/how-to/self-hosting.md :white_check_mark:Succeeded View
microsoft-edge/webview2/how-to/set-preview-channel.md :white_check_mark:Succeeded View
microsoft-edge/webview2/how-to/set-preview-channel-images/system-properties-env-vars.png :white_check_mark:Succeeded View
microsoft-edge/webview2/how-to/webdriver.md :white_check_mark:Succeeded View
microsoft-edge/webview2/samples/webview2_sample_uwp.md :white_check_mark:Succeeded View
microsoft-edge/webview2/samples/webview2apissample.md :white_check_mark:Succeeded View

For more details, please refer to the build report.

For any questions, please:

learn-build-service-prod[bot] commented 4 months ago

Learn Build status updates of commit e556e37:

:warning: Validation status: warnings

File Status Preview URL Details
microsoft-edge/webview2/how-to/self-hosting.md :warning:Warning View Details
microsoft-edge/toc.yml :white_check_mark:Succeeded View
microsoft-edge/webview2/concepts/distribution.md :white_check_mark:Succeeded View
microsoft-edge/webview2/concepts/versioning.md :white_check_mark:Succeeded View
microsoft-edge/webview2/how-to/prerelease-testing.md :white_check_mark:Succeeded View
microsoft-edge/webview2/how-to/set-preview-channel.md :white_check_mark:Succeeded View
microsoft-edge/webview2/how-to/set-preview-channel-images/system-properties-env-vars.png :white_check_mark:Succeeded View
microsoft-edge/webview2/how-to/webdriver.md :white_check_mark:Succeeded View
microsoft-edge/webview2/samples/webview2_sample_uwp.md :white_check_mark:Succeeded View
microsoft-edge/webview2/samples/webview2apissample.md :white_check_mark:Succeeded View

microsoft-edge/webview2/how-to/self-hosting.md

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

victorhuangwq commented 4 months ago

@champnic thanks for taking time to do a thorough review. I left the unaddressed comments opened and I will discuss them with @vickiez about it.

With regards to the overarching issue of the duplicate information when it comes to the instructions on how to change the channel search order, I agree there's definitely duplicated information - which can get difficult for us to maintain in the future.

The initial consideration is that we would prefer to keep information in the same page, as there's also slight variations in recommended option for the different articles.

I will take a look again on this aspect of it again once the preview learn.microsoft articles are fully rendered.

champnic commented 4 months ago

Sounds good, thanks @victorhuangwq!

Having it in one place can also highlight the subtle differences in recommendations, which I think can be beneficial. Otherwise as people read through they may see that it looks similar, and just assume it's the same info.

Overall I think I would strongly caution against duplication, but ultimately I'll let you and @vickiez decide how you want to structure this.

learn-build-service-prod[bot] commented 4 months ago

Learn Build status updates of commit d07a824:

:warning: Validation status: warnings

File Status Preview URL Details
microsoft-edge/webview2/how-to/self-hosting.md :warning:Warning View Details
microsoft-edge/toc.yml :white_check_mark:Succeeded View
microsoft-edge/webview2/concepts/distribution.md :white_check_mark:Succeeded View
microsoft-edge/webview2/concepts/versioning.md :white_check_mark:Succeeded View
microsoft-edge/webview2/how-to/prerelease-testing.md :white_check_mark:Succeeded View
microsoft-edge/webview2/how-to/set-preview-channel.md :white_check_mark:Succeeded View
microsoft-edge/webview2/how-to/set-preview-channel-images/system-properties-env-vars.png :white_check_mark:Succeeded View
microsoft-edge/webview2/how-to/webdriver.md :white_check_mark:Succeeded View
microsoft-edge/webview2/samples/webview2_sample_uwp.md :white_check_mark:Succeeded View
microsoft-edge/webview2/samples/webview2apissample.md :white_check_mark:Succeeded View

microsoft-edge/webview2/how-to/self-hosting.md

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

learn-build-service-prod[bot] commented 4 months ago

Learn Build status updates of commit 8c8d1bd:

:warning: Validation status: warnings

File Status Preview URL Details
microsoft-edge/webview2/how-to/self-hosting.md :warning:Warning View Details
microsoft-edge/toc.yml :white_check_mark:Succeeded View
microsoft-edge/webview2/concepts/distribution.md :white_check_mark:Succeeded View
microsoft-edge/webview2/concepts/versioning.md :white_check_mark:Succeeded View
microsoft-edge/webview2/how-to/prerelease-testing.md :white_check_mark:Succeeded View
microsoft-edge/webview2/how-to/set-preview-channel.md :white_check_mark:Succeeded View
microsoft-edge/webview2/how-to/set-preview-channel-images/system-properties-env-vars.png :white_check_mark:Succeeded View
microsoft-edge/webview2/how-to/webdriver.md :white_check_mark:Succeeded View
microsoft-edge/webview2/samples/webview2_sample_uwp.md :white_check_mark:Succeeded View
microsoft-edge/webview2/samples/webview2apissample.md :white_check_mark:Succeeded View

microsoft-edge/webview2/how-to/self-hosting.md

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

learn-build-service-prod[bot] commented 4 months ago

Learn Build status updates of commit 7798b65:

:white_check_mark: Validation status: passed

File Status Preview URL Details
microsoft-edge/toc.yml :white_check_mark:Succeeded View
microsoft-edge/webview2/concepts/developer-guide.md :white_check_mark:Succeeded View
microsoft-edge/webview2/concepts/distribution.md :white_check_mark:Succeeded View
microsoft-edge/webview2/concepts/versioning.md :white_check_mark:Succeeded View
microsoft-edge/webview2/get-started/winui.md :white_check_mark:Succeeded View
microsoft-edge/webview2/get-started/wpf.md :white_check_mark:Succeeded View
microsoft-edge/webview2/how-to/machine-setup.md :white_check_mark:Succeeded View
microsoft-edge/webview2/how-to/prerelease-testing.md :white_check_mark:Succeeded View
microsoft-edge/webview2/how-to/self-hosting.md :white_check_mark:Succeeded View
microsoft-edge/webview2/how-to/set-preview-channel.md :white_check_mark:Succeeded View
microsoft-edge/webview2/how-to/set-preview-channel-images/system-properties-env-vars.png :white_check_mark:Succeeded View
microsoft-edge/webview2/how-to/webdriver.md :white_check_mark:Succeeded View
microsoft-edge/webview2/index.md :white_check_mark:Succeeded View
microsoft-edge/webview2/samples/webview2_sample_uwp.md :white_check_mark:Succeeded View
microsoft-edge/webview2/samples/webview2apissample.md :white_check_mark:Succeeded View
microsoft-edge/webview2/samples/webview2browser.md :white_check_mark:Succeeded View

For more details, please refer to the build report.

For any questions, please: