You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@apisix.apache.org by GitBox <gi...@apache.org> on 2021/10/11 09:55:16 UTC

[GitHub] [apisix-dashboard] bzp2010 commented on pull request #2099: feat: support proto API

bzp2010 commented on pull request #2099:
URL: https://github.com/apache/apisix-dashboard/pull/2099#issuecomment-939872213


   ## Update
   - Some changes on the store interface have been split and merged. #2156 #2158 #2160
   - The entity part has been merged. #2144 
   - **An attempt was made to split the API handler, but this was difficult, so it was decided to still use this PR.**
   
   ## Code structure
   - api/internal/handler/proto/proto.go API handler
   - api/internal/route.go register route
   - api/internal/handler/proto/proto_test.go unit test
   - api/test/e2enew/proto/proto_test.go E2E test
   - api/test/e2enew/proto/proto_suite_test.go E2E test suite
   
   ## Review
   I have reorganized this PR, which only contains the relevant code of proto API. You can focus on the specific implementation in handler.
   
   ## cc
   cc @nic-chen @starsz 


-- 
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: notifications-unsubscribe@apisix.apache.org

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