- Issue created by @berdir
- 🇮🇳India kulpratap2002
I can't reproduce this issue because the routing.yml file contains only one route for the settings form, and I can't find any other separate settings form with a route.
- 🇨🇭Switzerland berdir Switzerland
Yes, and that one settings form is the problem. This module doesn't have separate settings form. It doesn't work, MonitoringPrometheusSettingsForm doesn't exist.
- 🇮🇳India kulpratap2002
@berdir Thanks for your reply.
If MonitoringPrometheusSettingsForm doesn't exist and there is no route defined for it, I can only find a single route for MonitoringSettingsForm and if we remove that then setting form doesn't show. Considering the issue summary mentions removing the broken route, what should be our next step?
- 🇨🇭Switzerland berdir Switzerland
The other route is dynamically defined in the the routing event subscriber. The next step is removing routing.yml completely.
- 🇮🇳India kulpratap2002
@berdir i have removed the broken route from monitoring_prometheus.routing.yml
Please review
Thank You -
berdir →
committed 589ae609 on 8.x-1.x authored by
kul.pratap →
Issue #3495372 by kul.pratap, berdir: Remove broken settings route
-
berdir →
committed 589ae609 on 8.x-1.x authored by
kul.pratap →
Automatically closed - issue fixed for 2 weeks with no activity.