[frontend] migrate/refactor StixCoreObjectKnowledgeBar component #9817
Conversation
github-actions
bot
added
the
filigran team
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #9817 +/- ##
==========================================
- Coverage 64.29% 64.28% -0.02%
==========================================
Files 655 655
Lines 62909 62909
Branches 6966 6962 -4
==========================================
- Hits 40450 40443 -7
- Misses 22459 22466 +7 ☔ View full report in Codecov by Sentry. |
delemaf
changed the title
|
I'm not sure if this problem is linked to this PR, but I didn't encounter it on testing or prerelease, so it might be related. As you see below, there's a large space on the left side of the knowledge bar : The knowledge bar also is narrower in comparison with testing. The problem must be due to that. |
| to={`${stixCoreObjectLink}/${path}`} | ||
| iconType={iconType} | ||
| label={label} | ||
| count={count} |
There was a problem hiding this comment.
Is it possible to update the counts for each item without refreshing the page easily? (with a refetch or a state)
There was a problem hiding this comment.
Not yet, the cause is probably the buggy relay query in the Root components and the way the subscription is done, even the counter of the table is not refreshed, it's not due to the refactor of this component. But I'll open a issue to handle this bug 🙏
delemaf
force-pushed
the
oob/migrate-knowledge-bar
branch
from
a4a4e8a to
42d4e08
Compare
...opencti-front/src/private/components/common/stix_core_objects/StixCoreObjectKnowledgeBar.tsx
Outdated
Show resolved
Hide resolved
...opencti-front/src/private/components/common/stix_core_objects/StixCoreObjectKnowledgeBar.jsx
Show resolved
Hide resolved
Proposed changes
Related issues
Checklist
Further comments