-
Couldn't load subscription status.
- Fork 265
implemented Deployment Configuration UI + backend #3220
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
base: feature/deployment_config
Are you sure you want to change the base?
implemented Deployment Configuration UI + backend #3220
Conversation
libs/dao/src/main/java/com/akto/dto/deployment/DeploymentConfig.java
Outdated
Show resolved
Hide resolved
apps/dashboard/src/main/java/com/akto/action/settings/DeploymentConfigAction.java
Show resolved
Hide resolved
apps/dashboard/src/main/java/com/akto/action/settings/DeploymentConfigAction.java
Outdated
Show resolved
Hide resolved
...eb/polaris_web/web/src/apps/dashboard/pages/settings/deployment_config/DeploymentConfigs.jsx
Outdated
Show resolved
Hide resolved
...eb/polaris_web/web/src/apps/dashboard/pages/settings/deployment_config/DeploymentConfigs.jsx
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR implements a comprehensive Deployment Configuration feature that allows users to manage environment variables for different deployments through both backend and frontend interfaces.
- Adds backend data models and DAO layer for deployment configurations with environment variable management
- Implements RESTful API endpoints for CRUD operations on deployment configs and environment variables
- Creates a full-featured React UI for managing deployment configurations with modals for adding, editing, and deleting environment variables
Reviewed Changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
TestingEndpoints.java |
Adds duplicate TAGS case in switch statement |
DeploymentConfig.java |
New DTO class for deployment configuration with nested EnvVariable class |
DeploymentConfigDao.java |
New DAO class with methods for deployment config and environment variable operations |
App.js |
Adds routing for deployment configs page |
SettingsLeftNav.jsx |
Adds navigation item for deployment configs |
api.js |
Frontend API client for deployment config operations |
DeploymentConfigs.jsx |
Main React component for deployment configuration management UI |
struts.xml |
Defines action mappings for all deployment config API endpoints |
DeploymentConfigAction.java |
Action class implementing all deployment config operations |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| import com.akto.dto.ApiInfo.ApiInfoKey; | ||
| import com.akto.dto.CollectionConditions.MethodCondition; | ||
| import com.akto.dto.testing.TagsTestingEndpoints; | ||
| import com.akto.dto.testing.TagsTestingEndpoints; |
Copilot
AI
Sep 23, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Duplicate import statement for TagsTestingEndpoints. This import already exists on line 7.
| import com.akto.dto.testing.TagsTestingEndpoints; |
| case TAGS: | ||
| String q = (data != null) ? data.getString("query") : null; | ||
| condition = new TagsTestingEndpoints(operator, q); | ||
| break; |
Copilot
AI
Sep 23, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Duplicate case for TAGS enum value. This exact case block already exists above with identical logic, making this code unreachable.
| case TAGS: | |
| String q = (data != null) ? data.getString("query") : null; | |
| condition = new TagsTestingEndpoints(operator, q); | |
| break; |
No description provided.