Conversation
Artuomka
deleted the
backend_remove_dashboard_id_from_generate_panel_position
branch
There was a problem hiding this comment.
Pull request overview
This pull request refactors the route configuration for the AI widget generation endpoint by removing the dashboardId parameter from the URL path. The change aligns the route structure with the endpoint's actual functionality - generating widget configurations without persisting them to a specific dashboard.
Changes:
- Updated the authentication middleware route configuration to change
/dashboard/:dashboardId/widget/generate/:connectionIdto/widget/generate/:connectionId
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| .forRoutes( | ||
| { path: '/dashboard/:dashboardId/widget/:connectionId', method: RequestMethod.POST }, | ||
| { path: '/dashboard/:dashboardId/widget/generate/:connectionId', method: RequestMethod.POST }, | ||
| { path: '/widget/generate/:connectionId', method: RequestMethod.POST }, |
There was a problem hiding this comment.
The route path has been changed from /dashboard/:dashboardId/widget/generate/:connectionId to /widget/generate/:connectionId, but the frontend code at frontend/src/app/services/dashboards.service.ts:149 still references the old path. This will cause API calls from the frontend to fail with 404 errors. The frontend code must be updated to use the new route path /widget/generate/${connectionId} instead of /dashboard/${dashboardId}/widget/generate/${connectionId}.
No description provided.