refactor(video): clean up unused imports and hooks in video components
Remove unused imports and hooks from VideoPage and useOpenAIVideo Simplify useOpenAIVideo by removing unnecessary effect and dependencies
This commit is contained in:
@@ -1,12 +1,9 @@
|
||||
import { retrieveVideo } from '@renderer/services/ApiService'
|
||||
import { SystemProviderIds } from '@renderer/types'
|
||||
import { useEffect } from 'react'
|
||||
import useSWR, { SWRConfiguration, useSWRConfig } from 'swr'
|
||||
|
||||
import { useProvider } from '../useProvider'
|
||||
import { useAddOpenAIVideo } from './useAddOpenAIVideo'
|
||||
import { useVideo } from './useVideo'
|
||||
import { useVideos } from './useVideos'
|
||||
|
||||
export const useOpenAIVideo = (id: string) => {
|
||||
const providerId = SystemProviderIds.openai
|
||||
@@ -19,8 +16,6 @@ export const useOpenAIVideo = (id: string) => {
|
||||
})
|
||||
}
|
||||
const video = useVideo(providerId, id)
|
||||
const { updateVideo } = useVideos(providerId)
|
||||
const addOpenAIVideo = useAddOpenAIVideo(providerId)
|
||||
let options: SWRConfiguration = {}
|
||||
switch (video?.status) {
|
||||
case 'queued':
|
||||
@@ -39,19 +34,6 @@ export const useOpenAIVideo = (id: string) => {
|
||||
const { mutate } = useSWRConfig()
|
||||
const revalidate = () => mutate(`video/openai/${id}`)
|
||||
|
||||
useEffect(() => {
|
||||
// queue -> in_progress / update progress
|
||||
if (data) {
|
||||
if (data.video.status === 'in_progress' && data.video.progress) {
|
||||
if (video) {
|
||||
updateVideo({ id: video.id, progress: data.video.progress })
|
||||
} else {
|
||||
addOpenAIVideo(data.video, 'Prompt lost')
|
||||
}
|
||||
}
|
||||
}
|
||||
}, [addOpenAIVideo, data, updateVideo, video])
|
||||
|
||||
return {
|
||||
video: data,
|
||||
isLoading,
|
||||
|
||||
@@ -3,6 +3,7 @@
|
||||
import { Divider } from '@heroui/react'
|
||||
import { Navbar, NavbarCenter } from '@renderer/components/app/Navbar'
|
||||
import { useProvider } from '@renderer/hooks/useProvider'
|
||||
import { useVideos } from '@renderer/hooks/video/useVideos'
|
||||
import { SystemProviderIds } from '@renderer/types'
|
||||
import { CreateVideoParams } from '@renderer/types/video'
|
||||
import { deepUpdate } from '@renderer/utils/deepUpdate'
|
||||
@@ -17,7 +18,6 @@ import { ProviderSetting } from './settings/ProviderSetting'
|
||||
import { SettingsGroup } from './settings/shared'
|
||||
import { VideoList } from './VideoList'
|
||||
import { VideoPanel } from './VideoPanel'
|
||||
import { useVideos } from '@renderer/hooks/video/useVideos'
|
||||
|
||||
export const VideoPage = () => {
|
||||
const { t } = useTranslation()
|
||||
|
||||
Reference in New Issue
Block a user