Vacant2333

Results 64 comments of Vacant2333

hi! after u deal with the ci, i will help u review! @Affan-7

@XiShanYongYe-Chang looks good to me /lgtm

@zhzhuang-zju i will help u to review after the ci pass~~~

(◦˙▽˙◦)

your route was not include `nginx_web/mainpage`, u must add a new route to include it

i think its not APISIX's problem, if the link was in the route so it will be recongnised by apisix

> Ummm, I can catchup your point. But if the link is at the same namespace, ex: `target.com/mainpage` --> `target.com/subpage` Do you mean I shall set up two routes for...

agree! for now, we may can use like `ENV_DOCKER_COMPOSE=docker compose .... make ...`

@shreemaan-abhishek how do u think? i agree with him, we may can update this