-
Notifications
You must be signed in to change notification settings - Fork 78
feat(FR-1407): add modify and delete action buttons for deployment detail page #4187
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: feat_FR-1388_/deployment-detail-page
Are you sure you want to change the base?
feat(FR-1407): add modify and delete action buttons for deployment detail page #4187
Conversation
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
How to use the Graphite Merge QueueAdd either label to this PR to merge it via the merge queue:
You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has required the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. This stack of pull requests is managed by Graphite. Learn more about stacking. |
Coverage report for
|
St.❔ |
Category | Percentage | Covered / Total |
|---|---|---|---|
| 🔴 | Statements | 4.34% (-0.02% 🔻) |
505/11635 |
| 🔴 | Branches | 3.45% (-0.01% 🔻) |
285/8255 |
| 🔴 | Functions | 2.43% (-0.01% 🔻) |
90/3706 |
| 🔴 | Lines | 4.31% (-0.02% 🔻) |
491/11394 |
Show new covered files 🐣
St.❔ |
File | Statements | Branches | Functions | Lines |
|---|---|---|---|---|---|
| 🔴 | ... / DeploymentModifyModal.tsx |
0% | 0% | 0% | 0% |
Test suite run success
114 tests passing in 13 suites.
Report generated by 🧪jest coverage report action from 0fcddbd
Coverage report for
|
St.❔ |
Category | Percentage | Covered / Total |
|---|---|---|---|
| 🔴 | Statements | 51.38% | 130/253 |
| 🔴 | Branches | 30.3% | 80/264 |
| 🔴 | Functions | 34.48% | 20/58 |
| 🔴 | Lines | 53.85% | 119/221 |
Test suite run success
55 tests passing in 3 suites.
Report generated by 🧪jest coverage report action from 3214332
7fe74b0 to
ceaccc5
Compare
757944c to
b1c4141
Compare
ceaccc5 to
c5c9eb9
Compare
b1c4141 to
5126e81
Compare
c5c9eb9 to
ab00f53
Compare
5126e81 to
27de565
Compare
ab00f53 to
ad4568f
Compare
27de565 to
73b7d3d
Compare
73b7d3d to
cec5b54
Compare
ad4568f to
049546a
Compare
cec5b54 to
7c25b49
Compare
049546a to
67fc52c
Compare
7c25b49 to
d4be88c
Compare
67fc52c to
1eded45
Compare
1eded45 to
0fcddbd
Compare
d4be88c to
c7ae034
Compare
0fcddbd to
798a0c9
Compare
c7ae034 to
7dd6f03
Compare
798a0c9 to
1e43668
Compare
7dd6f03 to
e8da4f8
Compare
1e43668 to
b703896
Compare
e8da4f8 to
9fb9227
Compare
b703896 to
da8f467
Compare
9fb9227 to
d5645ed
Compare
da8f467 to
9eabd3c
Compare
d5645ed to
cf9cd11
Compare
9eabd3c to
9bd309a
Compare
cf9cd11 to
812f1ab
Compare
9bd309a to
3214332
Compare

Resolves #4187 (FR-1407)
Summary
Changes
This feature enables users to modify deployment configurations and delete deployments directly from the detail page interface.