Skip to content

feat: add inline notification component #670

feat: add inline notification component

feat: add inline notification component #670

Triggered via pull request July 25, 2024 17:06
Status Failure
Total duration 3m 28s
Artifacts

build.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

2 errors and 3 warnings
build: player/react/src/lib/components/ui-inline-notification.tsx#L76
Property 'iconDescription' does not exist on type 'InlineNotificationState'.
build
Process completed with exit code 1.
lint
The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/checkout@v3, actions/setup-node@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
lint: sdk/react/src/lib/components/settings-context-pane.tsx#L211
React Hook useEffect has a missing dependency: 'setExternalSettings'. Either include it or remove the dependency array. If 'setExternalSettings' changes too often, find the parent component that defines it and wrap that definition in useCallback
build
The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/checkout@v3, actions/setup-node@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/