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

Improve distibution visualization #466

Closed

Conversation

bartoval
Copy link
Member

Description

Briefly describe the changes made in this Pull Request.

Additional context (optional)

Add any additional context or information about the changes made in this PR.

How to test

Provide clear and concise instructions on how to test the changes made in this PR.

bartoval and others added 30 commits August 30, 2024 01:06
refactor(Metrics): ♻️ Update metrics
fix(General): 🐞 Associate proxy host to the connector
…t_server

fix(Shared): 🐞 Processes and metrics in flowpairs details…
…ice_detail

refactor(Services): ♻️ Adjust service byterate details
fix(Metrics): 🐞 Wrong open connection calculation
…e_topology

fix(Topology): 🐞 Nodes redirect to unefined details when …
refactor(Component): ♻️ Create processList and overview compon…
…ress_process_Api

refactor(General): ♻️ Remove legacy address processes api
bartoval and others added 27 commits September 16, 2024 07:47
…ries

refactor(General): ♻️ Create hooks for data queries
refactor(Services): ♻️ Add isBound flag
…rammatically

refactor(General): ♻️ Hide table columns programmatically
…ithSiteIds

Remove bind links with site ids
@bartoval bartoval self-assigned this Sep 19, 2024
@bartoval bartoval closed this Sep 19, 2024
@codecov-commenter
Copy link

Codecov Report

Attention: Patch coverage is 95.06173% with 56 lines in your changes missing coverage. Please review.

Project coverage is 96.42%. Comparing base (e26ff41) to head (69868bf).

Files with missing lines Patch % Lines
...c/pages/Processes/components/ProcessBiFlowList.tsx 78.57% 18 Missing ⚠️
src/pages/Topology/components/NodeDetails.tsx 50.00% 8 Missing ⚠️
src/pages/Processes/components/PairsList.tsx 80.00% 6 Missing ⚠️
src/pages/Topology/services/index.ts 86.48% 5 Missing ⚠️
src/pages/ProcessGroups/hooks/useComponentsData.ts 80.00% 3 Missing ⚠️
src/pages/Topology/components/EdgeDetails.tsx 80.00% 3 Missing ⚠️
src/core/components/SkBiFlowList/index.tsx 86.66% 2 Missing ⚠️
src/pages/Sites/components/Details.tsx 75.00% 1 Missing and 1 partial ⚠️
src/pages/shared/Metrics/services/index.ts 95.91% 2 Missing ⚠️
...ore/components/SkBiFlowDetails/FlowDetailsBase.tsx 92.85% 1 Missing ⚠️
... and 6 more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #466      +/-   ##
==========================================
- Coverage   96.57%   96.42%   -0.16%     
==========================================
  Files         140      168      +28     
  Lines        3479     3576      +97     
  Branches      794      790       -4     
==========================================
+ Hits         3360     3448      +88     
- Misses        114      123       +9     
  Partials        5        5              

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants