Conversation
genwhittTTD
left a comment
genwhittTTD
left a comment
There was a problem hiding this comment.
Some copy edits plus please move the section to above the diagram and make it an h3. Thx much.
|
|
||
| | Port Number | Direction | Protocol | Description | | ||
| | ----------- | --------- | -------- | ------ | | ||
| | 80 | Inbound | HTTP | Serves all UID2 APIs, including the healthcheck endpoint `/ops/healthcheck`.<br/>When everything is up and running, the endpoint returns HTTP 200 with a response body of `OK`. For details, see [Running the Health Check](#running-the-health-check). | |
There was a problem hiding this comment.
healthcheck > health check
We use two words in UID2 docs (other than the actual endpoint name of course).
|
|
||
|  | ||
|
|
||
| #### Network Security Group Policy |
There was a problem hiding this comment.
@cYKatherine a key concern is the placement of this new content. In the existing doc there is a heading, https://unifiedid.com/docs/guides/operator-guide-azure-enclave#set-up-the-vpc-network, Set Up the VPC Network, and there is a diagram followed by steps. This new section has been added between the diagram and the steps, which is not correct.
In the AWS doc, which was mentioned in the thread (https://unifiedid.com/docs/guides/operator-guide-aws-marketplace#vpc-chart), it's right before the VPC chart, and at the same heading level. Therefore, for consistency, I think you should move this whole new section to above the "Set Up the VPC Network" heading (line 174 in the existing doc) and made it an h3 (three hashtags) rather than what you have which is an h4. That will be more parallel with the AWS doc.
Co-authored-by: Gen Whitt <107279666+genwhittTTD@users.noreply.github.com>
Co-authored-by: Gen Whitt <107279666+genwhittTTD@users.noreply.github.com>
3 participants
No description provided.