Yiyiyimu commented on a change in pull request #3980:
URL: https://github.com/apache/apisix/pull/3980#discussion_r607128231



##########
File path: docs/en/latest/getting-started.md
##########
@@ -216,7 +216,7 @@ $ curl http://127.0.0.1:9080/apisix/admin/consumers -H 
'X-API-KEY: edd1c9f034335
 Next, let's bind our `Consumer(John)` to that `Route`, we only need to 
**Enable** the [key-auth](./plugins/key-auth.md) plugin for that `Route`:
 
 ```bash
-curl http://127.0.0.1:9080/apisix/admin/routes/5 -H 'X-API-KEY: 
edd1c9f034335f136f87ad84b625c8f1' -X PUT -d '
+$ curl http://127.0.0.1:9080/apisix/admin/routes/5 -H 'X-API-KEY: 
edd1c9f034335f136f87ad84b625c8f1' -X PUT -d '

Review comment:
       Found some examples:
   - Kubernetes: do not have `$` 
(https://kubernetes.io/docs/tutorials/hello-minikube/)
   - Envoy: have `$` but the copy button does not fetch it 
(https://www.envoyproxy.io/docs/envoy/latest/start/quick-start/run-envoy)
   - Istio: same with envoy 
(https://istio.io/latest/docs/setup/getting-started/)
   
   So it seems the copy button could be customized




-- 
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to