Use the new Security Dashboard Layout in the Security Dashboard component
Summary
We're about to move the security dashboard to a GraphQL approach, however, parts of the application still need to use the Rest API. Therefore, as discussed here, we have created a security_dashboard_layout
component that will be used by both the GraphQL and Rest versions of the security dashboard.
This is issue is to update the current security_dashboard
component to use the new security_dashboard_layout
component and rename it to something like security_dashboard_rest
or (see below) security_dashboard_legacy
security_dashboard_vuex
.
Improvements
This approach lets us work simultaneously on the GraphQL migration, as while one person takes care of the GraphQL part, the other one works on refactoring the current component.
Risks
I'll quote @samdbeckham here:
I can't forsee any risks here, but they sound like famous last words
😅
Involved components
ee/app/assets/javascripts/security_dashboard/components/security_dashboard.vue
ee/app/assets/javascripts/security_dashboard/components/security_dashboard_layout.vue
Along with the relevant test files.