Skip to content
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

[backend/frontend] update endpoint list and add endpoint overview #2153

Merged
merged 79 commits into from
Jan 8, 2025

Conversation

savacano28
Copy link
Contributor

@savacano28 savacano28 commented Jan 2, 2025

Proposed changes

  • Update view of endpoint list (add column privilege, status, executor, refact enpoint popover, add pagination and filters)
  • Add endpoint overview with agent list page (creation composants agentPrivilege, agentDeploymentMode)
  • Move some files from obas framework to obas api

PR Doc: OpenBAS-Platform/docs#77

Related issues

Checklist

  • I consider the submitted work as finished
  • I tested the code for its functionality
  • I wrote test cases for the relevant uses case
  • I added/update the relevant documentation (either on github or on notion)
  • Where necessary I refactored code to improve the overall quality
  • For bug fix -> I implemented a test that covers the bug

Further comments

If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

@savacano28 savacano28 changed the base branch from master to release/1.11.0 January 2, 2025 14:36
@savacano28 savacano28 self-assigned this Jan 2, 2025
@savacano28 savacano28 added the filigran team use to identify PR from the Filigran team label Jan 2, 2025
@savacano28 savacano28 force-pushed the issue/2114 branch 3 times, most recently from 5003aa2 to 0a654eb Compare January 6, 2025 07:48
@savacano28 savacano28 marked this pull request as ready for review January 6, 2025 17:35
Copy link

codecov bot commented Jan 6, 2025

Codecov Report

Attention: Patch coverage is 51.51515% with 32 lines in your changes missing coverage. Please review.

Project coverage is 32.57%. Comparing base (673123e) to head (dd4ff3c).
Report is 1 commits behind head on release/1.11.0.

Files with missing lines Patch % Lines
...src/main/java/io/openbas/utils/EndpointMapper.java 55.55% 17 Missing and 3 partials ⚠️
...va/io/openbas/rest/asset/endpoint/EndpointApi.java 25.00% 6 Missing ⚠️
.../main/java/io/openbas/service/EndpointService.java 58.33% 5 Missing ⚠️
.../openbas/rest/atomic_testing/AtomicTestingApi.java 0.00% 1 Missing ⚠️
Additional details and impacted files
@@                 Coverage Diff                  @@
##             release/1.11.0    #2153      +/-   ##
====================================================
- Coverage             33.92%   32.57%   -1.36%     
- Complexity             1425     1489      +64     
====================================================
  Files                   546      574      +28     
  Lines                 16707    17925    +1218     
  Branches               1062     1134      +72     
====================================================
+ Hits                   5668     5839     +171     
- Misses                10798    11833    +1035     
- Partials                241      253      +12     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@savacano28 savacano28 merged commit 6fc2a1c into release/1.11.0 Jan 8, 2025
6 checks passed
@savacano28 savacano28 deleted the issue/2114 branch January 8, 2025 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
filigran team use to identify PR from the Filigran team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Front end UI part - agent installation logic
3 participants