-
Notifications
You must be signed in to change notification settings - Fork 93
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] remove option to remove dynamic asset from asset group (#2434) #2437
[frontend] remove option to remove dynamic asset from asset group (#2434) #2437
Conversation
Signed-off-by: Antoine MAZEAS <[email protected]>
Signed-off-by: Antoine MAZEAS <[email protected]>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## release/current #2437 +/- ##
=====================================================
- Coverage 38.29% 38.26% -0.03%
- Complexity 1809 1811 +2
=====================================================
Files 598 598
Lines 18403 18430 +27
Branches 1230 1231 +1
=====================================================
+ Hits 7047 7052 +5
- Misses 10971 10990 +19
- Partials 385 388 +3 ☔ View full report in Codecov by Sentry. |
Signed-off-by: Antoine MAZEAS <[email protected]>
}, [endpointIds]); | ||
|
||
const isLoading = loading && endpointIds.length > 0; | ||
|
||
return ( | ||
<> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
With your modification, you don't need this tags anymore
Signed-off-by: Antoine MAZEAS <[email protected]>
Proposed changes
Related issues
Checklist
Known remaining issues