Skip to content
JS Linting

Update Settings UI to unify the existing onboarding and settings experience. #28

Sign in to view logs

Update Settings UI to unify the existing onboarding and settings experience.

Update Settings UI to unify the existing onboarding and settings experience. #28

Triggered via pull request December 16, 2024 11:39
@iamdharmeshiamdharmesh
synchronize #824
enhancement/react-settings-ui-v2
Status Success
Total duration 49s
Artifacts –

eslint.yml

on: pull_request
eslint
42s
eslint
Fit to window
Zoom out
Zoom in

Annotations

3 warnings
eslint
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
eslint: src/js/settings/components/feature-additional-settings/classification-previewers/openai-embedding.js#L86
React Hook useEffect has missing dependencies: 'setIsPreviewerOpen' and 'setPreviewUnderProcess'. Either include them or remove the dependency array
eslint: src/js/settings/components/service-settings/index.js#L110
React Hook useEffect has a missing dependency: 'saveSettings'. Either include it or remove the dependency array