Open elipe17 opened 1 week ago
Attention: Patch coverage is 0%
with 2 lines
in your changes missing coverage. Please review.
Project coverage is 91.50%. Comparing base (
8d01df3
) to head (f30fdc2
). Report is 2 commits behind head on develop.
Files with missing lines | Patch % | Lines |
---|---|---|
...ackend/tdpservice/users/api/authorization_check.py | 0.00% | 2 Missing :warning: |
🚨 Try these New Features:
Summary of Changes
plg/deploy.sh
to deploy alertmanager and handle networkingHow to Test
cf add-network-policy alertmanager tdp-frontend-<ENV> --protocol tcp --port 80
andcf add-network-policy tdp-frontend-<ENV> alertmanager --protocol tcp --port 8080
where<ENV>
should be replaced withraft
, orqasp
, etc...Alerts
tab is visible and proxies the alertmanager UI when clickedDeliverables
More details on how deliverables herein are assessed included here.
Deliverable 1: Accepted Features
Checklist of ACs:
Deliverable 2: Tested Code
CodeCov Report
comment in PR)CodeCov Report
comment in PR)Deliverable 3: Properly Styled Code
Deliverable 4: Accessible
iamjolly
andttran-hub
using Accessibility Insights reveal any errors introduced in this PR?Deliverable 5: Deployed
Deliverable 6: Documented
Deliverable 7: Secure
Deliverable 8: User Research
Research product(s) clearly articulate(s):