Skip to content

test: add e2e tests for superset embedded dashboards feature #527

test: add e2e tests for superset embedded dashboards feature

test: add e2e tests for superset embedded dashboards feature #527

Re-run triggered January 31, 2025 11:31
Status Success
Total duration 3m 5s
Artifacts 1

verify-pr.yml

on: pull_request
call-workflow-e2e-prod  /  compute-prod-versions
3s
call-workflow-e2e-prod / compute-prod-versions
Matrix: call-workflow-e2e-prod / e2e-prod
call-workflow-e2e-prod  /  e2e-tests-success
0s
call-workflow-e2e-prod / e2e-tests-success
Fit to window
Zoom out
Zoom in

Annotations

9 warnings
lint: src/components/App.js#L44
React Hook useEffect has a missing dependency: 'props'. Either include it or remove the dependency array. However, 'props' will change when *any* prop changes, so the preferred fix is to destructure the 'props' object outside of the useEffect call and refer to those specific props inside useEffect
lint: src/pages/edit/EditDashboard.js#L56
React Hook useEffect has missing dependencies: 'dataEngine', 'props', and 'width'. Either include them or remove the dependency array. However, 'props' will change when *any* prop changes, so the preferred fix is to destructure the 'props' object outside of the useEffect call and refer to those specific props inside useEffect
lint: src/pages/edit/LayoutModal.js#L33
React Hook useEffect has a missing dependency: 'columns'. Either include it or remove the dependency array. If 'setCols' needs the current value of 'columns', you can also switch to useReducer instead of useState and read 'columns' in the reducer
lint: src/pages/edit/NewDashboard.js#L35
React Hook useEffect has missing dependencies: 'dispatch' and 'width'. Either include them or remove the dependency array
lint: src/pages/print/PrintDashboard.js#L105
React Hook useEffect has missing dependencies: 'addDashboardItem', 'dataEngine', 'removeDashboardItem', 'setPrintDashboard', and 'updateDashboardItem'. Either include them or remove the dependency array. If 'setPrintDashboard' changes too often, find the parent component that defines it and wrap that definition in useCallback
lint: src/pages/print/PrintLayoutDashboard.js#L92
React Hook useEffect has missing dependencies: 'customizePrintLayoutDashboard', 'dataEngine', 'id', and 'setPrintDashboard'. Either include them or remove the dependency array. If 'setPrintDashboard' changes too often, find the parent component that defines it and wrap that definition in useCallback
lint: src/pages/start/LandingPage.js#L9
React Hook useEffect has a missing dependency: 'onMount'. Either include it or remove the dependency array. If 'onMount' changes too often, find the parent component that defines it and wrap that definition in useCallback
lint: src/pages/start/StartScreen.js#L27
React Hook useEffect has a missing dependency: 'dataEngine'. Either include it or remove the dependency array
setup-matrix
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/