-
Notifications
You must be signed in to change notification settings - Fork 528
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: user can skip upstream when select service_id #1302
Conversation
just to make sure that this pr could also solves #1278 , if so I will move to other issues, thanks ^-^ |
Codecov Report
@@ Coverage Diff @@
## master #1302 +/- ##
==========================================
+ Coverage 44.36% 44.47% +0.11%
==========================================
Files 31 32 +1
Lines 1961 1965 +4
==========================================
+ Hits 870 874 +4
- Misses 979 980 +1
+ Partials 112 111 -1
Continue to review full report at Codecov.
|
web/cypress/integration/route/create-route-can-skip-upstream.spec.js
Outdated
Show resolved
Hide resolved
web/cypress/integration/route/create-route-can-skip-upstream.spec.js
Outdated
Show resolved
Hide resolved
web/cypress/integration/route/create-route-can-skip-upstream.spec.js
Outdated
Show resolved
Hide resolved
web/cypress/integration/route/create-route-can-skip-upstream.spec.js
Outdated
Show resolved
Hide resolved
web/cypress/integration/route/create-route-can-skip-upstream.spec.js
Outdated
Show resolved
Hide resolved
web/cypress/integration/route/create-route-can-skip-upstream.spec.js
Outdated
Show resolved
Hide resolved
Ping |
Please answer these questions before submitting a pull request
Why submit this pull request?
Bugfix
New feature provided
Improve performance
Backport patches
Related issues
resolve The process of creating routes using dashboard needs improvement #1213, resolve bug(fe): route created with service can not use the service upstream #1278
New feature or improvement
user can skip upstream when select service_id.
Please add the corresponding test cases if necessary.