[GitHub] [apisix-ingress-controller] juzhiyuan commented on a change in pull request #316: docs: tells users where they are

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

juzhiyuan commented on a change in pull request #316:
URL: https://github.com/apache/apisix-ingress-controller/pull/316#discussion_r602717684



##########
File path: docs/en/latest/config.json
##########
@@ -1,6 +1,11 @@
 {
   "version": "0.3.0",
   "sidebar": [
+    {
+      "type": "link",

Review comment:
       aha, a good way 😂

##########
File path: docs/en/latest/deployments/ack.md
##########
@@ -39,7 +39,6 @@ This document explains how to install Ingress APISIX on [ali-cloud ACK](https://
 cd /path/to/apisix-helm-chart
 helm repo add bitnami https://charts.bitnami.com/bitnami
 helm repo add apisix https://charts.apiseven.com
-# Use `hem search repo apisix` to search charts about apisix

Review comment:
       Why we need to remove this?

##########
File path: docs/en/latest/config.json
##########
@@ -1,6 +1,11 @@
 {
   "version": "0.3.0",
   "sidebar": [
+    {
+      "type": "link",
+      "label": "Apisix Ingress Controller",

Review comment:
       ```suggestion
         "label": "Apache APISIX Ingress Controller",
   ```




-- 
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:
users@infra.apache.org