Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

useVideo Causes Hot Reload Crash On Fresh Expo App #2878

Open
1 task done
vinceprofeta opened this issue Jan 9, 2025 · 4 comments
Open
1 task done

useVideo Causes Hot Reload Crash On Fresh Expo App #2878

vinceprofeta opened this issue Jan 9, 2025 · 4 comments
Labels
bug Something isn't working

Comments

@vinceprofeta
Copy link

Description

Hot reloading is causing an app crash with useVideo. No Error Logs. I have tried with a local asset using expo-asset still causing a crash on hot reload.

export default function HomeScreen() {
  const video = useVideo(
    'https://file-examples.com/storage/fe602ed48f677b2319947f8/2017/04/file_example_MP4_480_1_5MG.mp4',
    {
      looping: true,
    },
  )
  return <View/>
}

React Native Skia Version

^1.8.2

React Native Version

0.76.5

Using New Architecture

  • Enabled

Steps to Reproduce

Fresh Expo Install
Latest RN Skia Install
Implement useVideo hook
Add a log or text to force hot reload.
Crash

Snack, Code Example, Screenshot, or Link to Repository

export default function HomeScreen() {
  const video = useVideo(
    'https://file-examples.com/storage/fe602ed48f677b2319947f8/2017/04/file_example_MP4_480_1_5MG.mp4',
    {
      looping: true,
    },
  )
  return <View/>
}
@vinceprofeta vinceprofeta added the bug Something isn't working label Jan 9, 2025
@wcandillon
Copy link
Contributor

hello there 👋 react-native-skia-video might be better suited for you: https://github.com/AzzappApp/react-native-skia-video
We are aware of two strong packages that leverage Video + Skia and we think they'll do a better job at doing video than us. We might consider deprecating useVideo in the near future. I hope this helps.

@vinceprofeta
Copy link
Author

@wcandillon Thanks!

I spent a few hours trying to understand what was going on. Please add a note to the documentation.

@wcandillon wcandillon reopened this Jan 9, 2025
@wcandillon
Copy link
Contributor

Yes we will consolidate the situation in the short term regarding this.

@Nipun3120
Copy link

@wcandillon any eta on this ? i'm facing the same issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants