feat: react native video skill#11
Conversation
…-video # Conflicts: # README.md # skills/stream-react-native/builder.md
# Conflicts: # skills/stream-react-native/RULES.md # skills/stream-react-native/builder.md # skills/stream-react-native/sdk.md
isekovanic
left a comment
There was a problem hiding this comment.
Some slight remarks, I want to triple check the removed snippets but otherwise looks great !
|
|
||
| Required Android setup in the host app: | ||
|
|
||
| - `android/build.gradle`: `minSdkVersion = 24` |
There was a problem hiding this comment.
I think specific hardcoded min/target versions might not be future proof, shall we omit these ? create-react-native-app and create-expo-app handle this themselves I believe.
Or is this some WebRCT specific thing ?
There was a problem hiding this comment.
webrtc specific thing.. may get updated in the future.. but important to keep it here though, its a strict requirement for webrtc.. it is also in our video installtion docs
There was a problem hiding this comment.
But I will update to make sure to keep it 24 only if the minSDK version is already 24+, this seems to hardcode to 24 always
isekovanic
left a comment
There was a problem hiding this comment.
Double checked the snippets, they already belong to the blueprints spec ! They should've never been in builder.md/sdk.md at all. Thanks for the improvement !
Testing
Open a fresh Claude Code session inside any RN/Expo project (or just an empty directory for a new app)
Run either of the prompts below
Out of scope
Feeds RN, livestream/audio-room/theming/component-override blueprints are out of scope (parity with PR #8)