Consolidate HttpEnabled explaination #1014
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Changes
The HttpService page currently provides multiple explainers of HttpEnabled - but none under the HttpEnabled property. I've spoken to a few people who have found this confusing, particularly when being linked into this page and missing the summary at the top (especially given that the sub-heading for 'Enable HTTP requests' does not show on the 'On This Page' right-hand navigation.
This PR consolidates the explanation on the page under the HttpEnabled property, linking down to it from the summary.
If this PR is merged, then the code samples on this page should also be updated to just link to the HttpEnabled property as a single source-of-truth explainer. For example, the astronaut sample currently contains an explanation that doesn't consider setting with command bar, whilst the Pastebin and RequestAsync samples do not mention enabling requests at all.
(obviously I can't currently update the samples as they're not present on repo!)
Checks
By submitting your pull request for review, you agree to the following: