Skip to content

Conversation

dikjain
Copy link

@dikjain dikjain commented Oct 10, 2025

  • Implemented getAvailabilityInfo method in ExpoAVVideoAdapter and ExpoVideoAdapter to provide detailed error messages and installation commands if the required modules or components are not found.
  • Updated renderVideo methods to utilize the new availability checks.
  • Added a simple getAvailabilityInfo method in FallbackVideoAdapter for consistency.
  • Updated VideoPlayerAdapter interface to include the new method for availability information.

Brief Summary of Changes

Replaced vague "expo-av/expo-video is not available" error messages with clear, actionable error messages that include specific installation commands. The new getAvailabilityInfo() method provides structured error information including the missing package name, descriptive error message, and exact installation command to resolve the issue.

What does this PR address?

Are tests included?

  • Yes
  • No

Reviewer, please note:

nothing as such, since it's a simlple fix

  • This is a minimal, backwards-compatible change that only improves error messaging
  • The new getAvailabilityInfo() method follows the existing adapter pattern
  • All existing tests continue to pass without modification

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I ran the full test suite before pushing the changes and all the tests pass.

- Implemented `getAvailabilityInfo` method in `ExpoAVVideoAdapter` and `ExpoVideoAdapter` to provide detailed error messages and installation commands if the required modules or components are not found.
- Updated `renderVideo` methods to utilize the new availability checks.
- Added a simple `getAvailabilityInfo` method in `FallbackVideoAdapter` for consistency.
- Updated `VideoPlayerAdapter` interface to include the new method for availability information.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant