[GitHub] [apisix] Drery commented on pull request #5940: feat: mocking plugin

2022-02-17 Thread GitBox


Drery commented on pull request #5940:
URL: https://github.com/apache/apisix/pull/5940#issuecomment-1043987746


   @spacewander @leslie-tsang Thanks, I resolved the conflicts, please have a 
review again.


-- 
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-unsubscr...@apisix.apache.org

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




[GitHub] [apisix] Drery commented on pull request #5940: feat: mocking plugin

2022-02-17 Thread GitBox


Drery commented on pull request #5940:
URL: https://github.com/apache/apisix/pull/5940#issuecomment-1043731989


   @shuaijinchao @juzhiyuan @leslie-tsang @spacewander 
请各位大佬再看一下,还有不要再抠格式排版这种细节了好吗???


-- 
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-unsubscr...@apisix.apache.org

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




[GitHub] [apisix] Drery commented on pull request #5940: feat: mocking plugin

2022-02-15 Thread GitBox


Drery commented on pull request #5940:
URL: https://github.com/apache/apisix/pull/5940#issuecomment-1041074878


   @leslie-tsang done


-- 
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-unsubscr...@apisix.apache.org

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




[GitHub] [apisix] Drery commented on pull request #5940: feat: mocking plugin

2022-02-15 Thread GitBox


Drery commented on pull request #5940:
URL: https://github.com/apache/apisix/pull/5940#issuecomment-1041053914


   @spacewander done, is it sorted as this?


-- 
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-unsubscr...@apisix.apache.org

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




[GitHub] [apisix] Drery commented on pull request #5940: feat: mocking plugin

2022-02-15 Thread GitBox


Drery commented on pull request #5940:
URL: https://github.com/apache/apisix/pull/5940#issuecomment-1041029268


   @spacewander All comment resolved, please review again


-- 
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-unsubscr...@apisix.apache.org

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




[GitHub] [apisix] Drery commented on pull request #5940: feat: mocking plugin

2022-02-15 Thread GitBox


Drery commented on pull request #5940:
URL: https://github.com/apache/apisix/pull/5940#issuecomment-1039775513






-- 
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-unsubscr...@apisix.apache.org

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




[GitHub] [apisix] Drery commented on pull request #5940: feat: mocking plugin

2022-02-14 Thread GitBox


Drery commented on pull request #5940:
URL: https://github.com/apache/apisix/pull/5940#issuecomment-1039926998


   @spacewander @shuaijinchao hello, all CI is success, please review again


-- 
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-unsubscr...@apisix.apache.org

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




[GitHub] [apisix] Drery commented on pull request #5940: feat: mocking plugin

2022-02-14 Thread GitBox


Drery commented on pull request #5940:
URL: https://github.com/apache/apisix/pull/5940#issuecomment-1039775513


   @spacewander I deleted one more line of code,please help me to run the CI 
again


-- 
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-unsubscr...@apisix.apache.org

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




[GitHub] [apisix] Drery commented on pull request #5940: feat: mocking plugin

2022-02-13 Thread GitBox


Drery commented on pull request #5940:
URL: https://github.com/apache/apisix/pull/5940#issuecomment-1038727785


   @spacewander @bisakhmondal All comment resolved, please take a look if 
that's right!


-- 
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-unsubscr...@apisix.apache.org

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




[GitHub] [apisix] Drery commented on pull request #5940: feat: mocking plugin

2022-02-13 Thread GitBox


Drery commented on pull request #5940:
URL: https://github.com/apache/apisix/pull/5940#issuecomment-1038559716


   @spacewander is there any progress?
   https://user-images.githubusercontent.com/9653109/153789392-1ad529ab-6928-4ac9-ac75-369ab34db1dc.png;>
   


-- 
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-unsubscr...@apisix.apache.org

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




[GitHub] [apisix] Drery commented on pull request #5940: feat: mocking plugin

2022-02-07 Thread GitBox


Drery commented on pull request #5940:
URL: https://github.com/apache/apisix/pull/5940#issuecomment-1032185633


   @starsz Hello, all CI is pass, please check again if there are any issues 
that need to be fixed~


-- 
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-unsubscr...@apisix.apache.org

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




[GitHub] [apisix] Drery commented on pull request #5940: feat: mocking plugin

2022-02-07 Thread GitBox


Drery commented on pull request #5940:
URL: https://github.com/apache/apisix/pull/5940#issuecomment-1032131858


   > 
   
   @spacewander I merged the master and the issue is not fixed, I fix it in my 
code, please run the CI again!


-- 
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-unsubscr...@apisix.apache.org

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




[GitHub] [apisix] Drery commented on pull request #5940: feat: mocking plugin

2022-01-29 Thread GitBox


Drery commented on pull request #5940:
URL: https://github.com/apache/apisix/pull/5940#issuecomment-1025073291


   > Let's merge master to make CI pass?
   
   @spacewander done


-- 
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-unsubscr...@apisix.apache.org

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




[GitHub] [apisix] Drery commented on pull request #5940: feat: mocking plugin

2022-01-27 Thread GitBox


Drery commented on pull request #5940:
URL: https://github.com/apache/apisix/pull/5940#issuecomment-1023036387


   @spacewander @starsz hello, the error is still exists, please pay attention 
to this issue~
   
![image](https://user-images.githubusercontent.com/9653109/151335894-96ade14f-3229-459f-865f-c952637610ea.png)
   the link is: 
https://github.com/apache/apisix/runs/4963622807?check_suite_focus=true 


-- 
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-unsubscr...@apisix.apache.org

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




[GitHub] [apisix] Drery commented on pull request #5940: feat: mocking plugin

2022-01-27 Thread GitBox


Drery commented on pull request #5940:
URL: https://github.com/apache/apisix/pull/5940#issuecomment-1022943852


   @starsz hello, there's a error when run the unit test, but it's not produce 
by my code, please have a look at this.
   
![image](https://user-images.githubusercontent.com/9653109/151316180-60a5f035-4d00-478e-89df-71b1244961aa.png)
   it's should be produced by this comment:
   
![38A93318B25375AF7FA705DDB2B5C909](https://user-images.githubusercontent.com/9653109/151316266-983edce6-093c-4225-9c68-13e84c886fcc.jpg)
   


-- 
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-unsubscr...@apisix.apache.org

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




[GitHub] [apisix] Drery commented on pull request #5940: feat: mocking plugin

2022-01-26 Thread GitBox


Drery commented on pull request #5940:
URL: https://github.com/apache/apisix/pull/5940#issuecomment-1022882747


   > LGTM. But CI failed.
   @starsz hello,I fixed, please run the CI again
   


-- 
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-unsubscr...@apisix.apache.org

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




[GitHub] [apisix] Drery commented on pull request #5940: feat: mocking plugin

2022-01-25 Thread GitBox


Drery commented on pull request #5940:
URL: https://github.com/apache/apisix/pull/5940#issuecomment-1021898782


   @starsz hello,please check again!


-- 
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-unsubscr...@apisix.apache.org

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




[GitHub] [apisix] Drery commented on pull request #5940: feat: mocking plugin

2022-01-25 Thread GitBox


Drery commented on pull request #5940:
URL: https://github.com/apache/apisix/pull/5940#issuecomment-1020956521


   @bisakhmondal @starsz I have already resolve the conversations, please check 
again


-- 
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-unsubscr...@apisix.apache.org

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




[GitHub] [apisix] Drery commented on pull request #5940: feat: mocking plugin

2022-01-25 Thread GitBox


Drery commented on pull request #5940:
URL: https://github.com/apache/apisix/pull/5940#issuecomment-1020956521


   @bisakhmondal @starsz I have already resolve the conversations, please check 
again


-- 
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-unsubscr...@apisix.apache.org

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




[GitHub] [apisix] Drery commented on pull request #5940: feat: mocking plugin

2022-01-24 Thread GitBox


Drery commented on pull request #5940:
URL: https://github.com/apache/apisix/pull/5940#issuecomment-1019897972


   @juzhiyuan @spacewander @shuaijinchao @leslie-tsang @bisakhmondal 
   hello, all workflow is pass, please review the code if there is any issue to 
fix?


-- 
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-unsubscr...@apisix.apache.org

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




[GitHub] [apisix] Drery commented on pull request #5940: feat: mocking plugin

2022-01-23 Thread GitBox


Drery commented on pull request #5940:
URL: https://github.com/apache/apisix/pull/5940#issuecomment-1019458130


   @spacewander @leslie-tsang 
   hello, what does this error mains?
   
![image](https://user-images.githubusercontent.com/9653109/150674956-9b6be56b-017c-4350-861b-08fe78f06161.png)
   the workflow link is: 
https://github.com/apache/apisix/runs/4911836590?check_suite_focus=true


-- 
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-unsubscr...@apisix.apache.org

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




[GitHub] [apisix] Drery commented on pull request #5940: feat: mocking plugin

2022-01-20 Thread GitBox


Drery commented on pull request #5940:
URL: https://github.com/apache/apisix/pull/5940#issuecomment-1018215189


   > Hi @Drery, there have 2 conflicts in Docs, please resolve them :)
   
   emm, i have no access to resolve conflicts.


-- 
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-unsubscr...@apisix.apache.org

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




[GitHub] [apisix] Drery commented on pull request #5940: feat: mocking plugin

2022-01-20 Thread GitBox


Drery commented on pull request #5940:
URL: https://github.com/apache/apisix/pull/5940#issuecomment-1018213802


   > Hi @Drery, there have 2 conflicts in Docs, please resolve them :)
   
   


-- 
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-unsubscr...@apisix.apache.org

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




[GitHub] [apisix] Drery commented on pull request #5940: feat: mocking plugin

2022-01-20 Thread GitBox


Drery commented on pull request #5940:
URL: https://github.com/apache/apisix/pull/5940#issuecomment-1018213459


   > Hi @Drery, there have 2 conflicts in Docs, please resolve them :)
   
   @juzhiyu


-- 
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-unsubscr...@apisix.apache.org

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




[GitHub] [apisix] Drery commented on pull request #5940: feat: mocking plugin

2021-12-30 Thread GitBox


Drery commented on pull request #5940:
URL: https://github.com/apache/apisix/pull/5940#issuecomment-1003299223


   @shuaijinchao @juzhiyuan 
   Thank you.
   I already send a proposal to the mailing list.


-- 
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-unsubscr...@apisix.apache.org

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




[GitHub] [apisix] Drery commented on pull request #5940: feat: mocking plugin

2021-12-29 Thread GitBox


Drery commented on pull request #5940:
URL: https://github.com/apache/apisix/pull/5940#issuecomment-1002476664


   > Glad to know that, and I used mock.js to do mocking as well  Could 
@shuaijinchao help to guide @Drery if we need this kind of plugin?
   @juzhiyuan thank you for your attention.
   just in our work scene, if the api gateway support mock response, will 
improve frontend development efficiency.
   we can bind or unbind the mocking plugin to convert the mock response or 
real upstream response, and it's need not to modify js code.


-- 
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-unsubscr...@apisix.apache.org

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




[GitHub] [apisix] Drery commented on pull request #5940: feat: mocking plugin

2021-12-29 Thread GitBox


Drery commented on pull request #5940:
URL: https://github.com/apache/apisix/pull/5940#issuecomment-1002468169


   @juzhiyuan the kong's mocking plugin is support too simple.i design this 
plugin  is reference to mock.js which is closer to the scenario used for 
front-end developer.
   
   https://github.com/nuysoft/Mock/wiki/Mock.Random


-- 
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-unsubscr...@apisix.apache.org

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