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

fix: fix system status modal. #2755

Closed

Conversation

miettal
Copy link
Contributor

@miettal miettal commented Aug 21, 2024

Sorry past my commit has bug for making component list logic.
in this commit I use concat but it's not in-place method so this code doesn't work.
This PR rewrte concat to push.
2664b13#diff-18c3e87449e26c6e1913ede0123a3d5aa61209b14e6cfb97fac0c9614553c352R591

Copy link

codecov bot commented Aug 21, 2024

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #2755      +/-   ##
=============================================
- Coverage      55.89%   55.88%   -0.00%     
+ Complexity      8047     8044       -3     
=============================================
  Files           2077     2077              
  Lines          87948    87948              
  Branches        6494     6494              
=============================================
- Hits           49149    49144       -5     
- Misses         37115    37121       +6     
+ Partials        1684     1683       -1     

@Sn0w3y
Copy link
Contributor

Sn0w3y commented Aug 21, 2024

Hi @miettal

already done here:

#2738

@miettal
Copy link
Contributor Author

miettal commented Aug 21, 2024

Thank you!!

@miettal miettal closed this Aug 21, 2024
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