-
Notifications
You must be signed in to change notification settings - Fork 106
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
[frontend/backend] Show expectations traces (#564) #2598
Conversation
66032e4
to
7af9369
Compare
a9453bb
to
cc03404
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## release/current #2598 +/- ##
=====================================================
+ Coverage 40.44% 40.59% +0.15%
- Complexity 2068 2081 +13
=====================================================
Files 635 638 +3
Lines 19441 19518 +77
Branches 1323 1324 +1
=====================================================
+ Hits 7862 7924 +62
- Misses 11148 11161 +13
- Partials 431 433 +2 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
4c4635f
to
398a9ae
Compare
...bas-api/src/main/java/io/openbas/migration/V3_74__Add_table_injects_expectations_traces.java
Outdated
Show resolved
Hide resolved
...bas-model/src/main/java/io/openbas/database/repository/InjectExpectationTraceRepository.java
Outdated
Show resolved
Hide resolved
openbas-api/src/main/java/io/openbas/rest/expectation/ExpectationApi.java
Outdated
Show resolved
Hide resolved
...as-api/src/main/java/io/openbas/rest/inject_expectation_trace/InjectExpectationTraceApi.java
Outdated
Show resolved
Hide resolved
openbas-api/src/main/java/io/openbas/service/InjectExpectationService.java
Outdated
Show resolved
Hide resolved
openbas-api/src/main/java/io/openbas/service/InjectExpectationService.java
Show resolved
Hide resolved
return simpleCall(uri); | ||
}; | ||
|
||
export const getAlertLinksCount = (injectExpectationId: string, sourceId: string | undefined, expectationResultSourceType: string | undefined) => { |
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.
todo: sourceId can't be null for the back-end. So we need to be aligned here.
openbas-front/src/admin/components/atomic_testings/atomic_testing/ExpirationChip.tsx
Outdated
Show resolved
Hide resolved
openbas-front/src/admin/components/atomic_testings/atomic_testing/TargetResultsDetail.tsx
Show resolved
Hide resolved
...-front/src/admin/components/atomic_testings/atomic_testing/TargetResultsSecurityPlatform.tsx
Outdated
Show resolved
Hide resolved
...-front/src/admin/components/atomic_testings/atomic_testing/TargetResultsSecurityPlatform.tsx
Outdated
Show resolved
Hide resolved
...as-api/src/main/java/io/openbas/rest/inject_expectation_trace/InjectExpectationTraceApi.java
Show resolved
Hide resolved
ded0569
to
7ec815e
Compare
d192aa5
to
2dde387
Compare
38e2ae7
to
ef6b83f
Compare
Closes #564