[GitHub] [pulsar] Anonymitaet commented on a diff in pull request #16267: [improve][doc] Add info for health check CMD and fix incorrect REST API links

[GitBox <gi...@apache.org>]

Anonymitaet commented on code in PR #16267:
URL: https://github.com/apache/pulsar/pull/16267#discussion_r910514295


##########
site2/docs/admin-api-brokers.md:
##########
@@ -128,7 +128,7 @@ BrokerInfo(serviceUrl=broker1.use.org.com:8080)
 </TabItem>
 <TabItem value="REST API">
 
-{@inject: endpoint|GET|/admin/v2/brokers/leaderBroker?version=@pulsar:version_number@}
+{@inject: endpoint|GET|/admin/v2/brokers/leaderBroker|operation/getLeaderBroker?version=@pulsar:version_number@}

Review Comment:
   1. Do these links go to the specific REST API doc rather than the home page?
   2. Will you fix all the occurrences in `master`?  



##########
site2/website/versioned_sidebars/version-2.8.3-sidebars.json:
##########
@@ -384,7 +384,7 @@
         },
         {
           "type": "doc",
-          "id": "version-2.8.2/security-basic-auth"
+          "id": "version-2.8.3/security-basic-auth"

Review Comment:
   reminder: any other doc versions need this chapter?



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org