Conversation
There was a problem hiding this comment.
Pull request overview
This PR introduces front-end support for Proxy management, wiring new OpenAPI-generated proxy API/types into the application and adding Redux state + UI pages to list, create, view, update, and delete proxies.
Changes:
- Added OpenAPI models + API client for proxies and exposed them through the central API client.
- Implemented Redux slice/epics and DTO↔model transforms for proxy CRUD flows.
- Added Proxy list/detail/create UI (routing + sidebar entry) and shared proxy status formatting/color utilities with tests.
Reviewed changes
Copilot reviewed 26 out of 26 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| src/utils/proxy.ts | Adds status options plus status→label/color helpers for proxy UI. |
| src/utils/proxy.spec.tsx | Component-test coverage for proxy status formatting and color mapping. |
| src/types/user-interface.tsx | Adds widget lock keys used by proxy list/detail widgets. |
| src/types/proxies.ts | Introduces proxy-focused type aliases wrapping OpenAPI DTOs. |
| src/types/openapi/models/index.ts | Exports new proxy-related OpenAPI models (plus an extra import). |
| src/types/openapi/models/Resource.ts | Adds Proxies resource enum value for RBAC/menu gating. |
| src/types/openapi/models/ProxyUpdateRequestDto.ts | New OpenAPI DTO for updating a proxy. |
| src/types/openapi/models/ProxyStatus.ts | New OpenAPI enum defining proxy lifecycle/status values. |
| src/types/openapi/models/ProxyRequestDto.ts | New OpenAPI DTO for creating a proxy. |
| src/types/openapi/models/ProxyListDto.ts | New OpenAPI DTO for listing proxies. |
| src/types/openapi/models/ProxyInstallInstructionsDto.ts | New OpenAPI DTO for installation instructions. |
| src/types/openapi/models/ProxyDto.ts | New OpenAPI DTO for full proxy details. |
| src/types/openapi/apis/index.ts | Exports the new ProxyManagementApi. |
| src/types/openapi/apis/ProxyManagementApi.ts | Adds generated API methods for proxy CRUD/list/instructions. |
| src/ducks/transform/proxies.ts | Adds simple transform helpers between OpenAPI DTOs and app models. |
| src/ducks/reducers.ts | Registers the new proxies slice reducer. |
| src/ducks/proxies.ts | Adds Redux slice/state/selectors/actions for proxies. |
| src/ducks/proxies-epic.ts | Adds epics to call proxy API client and coordinate redirects/locks/alerts. |
| src/ducks/index.ts | Registers proxy epics in the root epic combiner. |
| src/components/_pages/proxies/list/ProxiesList.tsx | Adds proxy list UI with status filter, create modal, and multi-select delete. |
| src/components/_pages/proxies/form/ProxyForm.tsx | Adds create-proxy form dialog content (name/description). |
| src/components/_pages/proxies/detail/ProxyDetail.tsx | Adds proxy detail view with inline description editing and delete. |
| src/components/_pages/proxies/ProxyStatusBadge.tsx | Adds a badge rendering status with consistent label/color mapping. |
| src/components/Layout/Sidebar/index.tsx | Adds “Proxies” navigation item gated by Resource.Proxies. |
| src/components/AppRouter.tsx | Adds routes for proxy list and detail pages. |
| src/api.ts | Wires the new proxy API client into backendClient / ApiClients. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 26 out of 26 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 26 out of 26 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 34 out of 34 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
src/components/InstallationInstructions/InstallationInstructions.spec.tsx
Outdated
Show resolved
Hide resolved
ivosh
force-pushed
the
feat/proxies
branch
2 times, most recently
from
4afa5be to
014e499
Compare
…tor into feat/proxies
…tor into feat/proxies
|
No description provided.