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 2020/11/03 13:48:26 UTC
[GitHub] [apisix-dashboard] LiteSun commented on a change in pull request #666: fix: promethus incorrect value when update route
LiteSun commented on a change in pull request #666:
URL: https://github.com/apache/apisix-dashboard/pull/666#discussion_r516381892
##########
File path: frontend/src/pages/Route/transform.ts
##########
@@ -152,10 +152,6 @@ export const transformRouteData = (data: RouteModule.Body) => {
const { plugins, script } = data;
- if (plugins && plugins.prometheus) {
- plugins.prometheus = { enabled: true };
Review comment:
> If we remove this line, and just enabled this plugin, then how about this plugin's status when we use Edit mode? Is it still enabled?
When we in Edit mode, API will return `plugins:{promethues:{}}`if we enabled it before and it works as expected.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org