-
Notifications
You must be signed in to change notification settings - Fork 941
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] Add button for selecting relationships in knowledge graph #8447
[frontend] Add button for selecting relationships in knowledge graph #8447
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## release/6.4.0 #8447 +/- ##
=================================================
- Coverage 66.23% 66.21% -0.02%
=================================================
Files 597 597
Lines 60300 60424 +124
Branches 6196 6204 +8
=================================================
+ Hits 39939 40010 +71
- Misses 20361 20414 +53
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
opencti-platform/opencti-front/src/private/components/analyses/reports/ReportKnowledgeGraph.jsx
Outdated
Show resolved
Hide resolved
opencti-platform/opencti-front/src/private/components/analyses/reports/ReportKnowledgeGraph.jsx
Outdated
Show resolved
Hide resolved
opencti-platform/opencti-front/src/private/components/analyses/reports/ReportKnowledgeGraph.jsx
Outdated
Show resolved
Hide resolved
Thanks for your PR ! 👍 |
4127a57
to
7d5533d
Compare
Can you also add the translations for the tooltips you added ? ie:
|
I used Google Translate; I hope this is okay. Also, changed the button position as requested. |
Thanks for the changes @stefan1anuby ! Your 2 last commits are not signed so I can't merge the PR... Can you sign them? Thanks |
…ild/parent) from selected nodes (OpenCTI-Platform#8378) Signed-off-by: stefan1anuby <[email protected]>
…Platform#8378) Signed-off-by: stefan1anuby <[email protected]>
ba3423b
to
5d7da31
Compare
Done! Thank you for the guidance! |
Proposed changes
Added a button to select/deselect relationships (children/parent/both) from or to selected nodes in a knowledge graph context.
2024-09-22.20-03-13.mp4
I'm not entirely sure about the icon choices and would appreciate feedback on UX/UI.
Related issues
part of #8378