Hi I have changed my pull request.
Please review it again. Thanks. > On Mar 4, 2022, at 2:03 PM, JunXu Chen <chenju...@apache.org> wrote: > > hi @吴治国 > > Thanks for your contribution. > > I commented on your question in the issue, please take a look. > > On Fri, 4 Mar 2022 at 11:45, 吴治国 <wzg547228...@163.com> wrote: > >> Thanks >> >> >>> On Mar 4, 2022, at 10:29 AM, Zhiyuan Ju <juzhiy...@apache.org> wrote: >>> >>> Hi, >>> >>> There have some comments on this issue, and we're seeking help from Xiran >>> Liu and Junxu Chen, they're more familiar with this. >>> >>> Best Regards! >>> @ Zhiyuan Ju <https://github.com/juzhiyuan> >>> >>> >>> 吴治国 <wzg547228...@163.com> 于2022年3月3日周四 17:19写道： >>> >>>> Hi Community >>>> I found a bug in dashboard project, when I had a route with many >>>> uri configuration, more than one route will be create after I export the >>>> OpenApi and import it. I try to fix it use merge route by name, but >>>> comments suggest to use id, but if we use id, there is many code should >> be >>>> change, both export and import interface. Which way should we choose? >>>> >>>> Issue: https://github.com/apache/apisix-dashboard/issues/2329 < >>>> https://github.com/apache/apisix-dashboard/issues/2329> >>>> PR: https://github.com/apache/apisix-dashboard/pull/2330 < >>>> https://github.com/apache/apisix-dashboard/pull/2330> >>>> >>>> >> >>
