Skip to content

Conversation

gabrieldonadel
Copy link
Contributor

Summary:

Trying to use react-native-slider with react-native 0.81.0-rc.5 will cause an insta crash in the app due to an issue when parsing RNCSliderProps on Android

image

These changes are related to the Props 2.0 project and require us to set the necessary flags and definitions in Cmake, by using target_compile_reactnative_options this can be done automatically instead of setting them manually.

Test Plan:

  1. npx @react-native-community/cli@latest init RC5 --skip-install --version next
  2. yarn add @react-native-community/slider
  3. patch package/android/src/main/jni/CMakeLists.txt

gabrieldonadel added a commit to expo/expo that referenced this pull request Aug 6, 2025
# Why

We should bump @react-native-community/slider to the latest version for
SDK 54

(waiting for callstack/react-native-slider#722
to be released in order to remove our patch)

# How

Update @react-native-community/slider to 5.0.0

# Test Plan

- BareExpo
- ExpoGo

# Checklist

<!--
Please check the appropriate items below if they apply to your diff.
-->

- [ ] I added a `changelog.md` entry and rebuilt the package sources
according to [this short
guide](https://github.com/expo/expo/blob/main/CONTRIBUTING.md#-before-submitting)
- [ ] This diff will work correctly for `npx expo prebuild` & EAS Build
(eg: updated a module plugin).
- [ ] Conforms with the [Documentation Writing Style
Guide](https://github.com/expo/expo/blob/main/guides/Expo%20Documentation%20Writing%20Style%20Guide.md)
Copy link
Member

@okwasniewski okwasniewski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Pining @BartoszKlonowski, @draggie

@gabrieldonadel
Copy link
Contributor Author

Can we merge this before the 0.81 release?

@okwasniewski okwasniewski merged commit f1ea7bb into callstack:main Aug 11, 2025
6 checks passed
@okwasniewski
Copy link
Member

@BartoszKlonowski can you release a new version with this fix? 🙏

@janodetzel
Copy link

Hi, we currently can't upgrade to RN 0.81 because of this issue. Is there an estimate for a new release with this fix?

gabrieldonadel added a commit to expo/expo that referenced this pull request Aug 19, 2025
# Why

callstack/react-native-slider#722 just got
released fixing support for 0.81, and we can now remove our patch
 
# How

Update @react-native-community/slider to 5.0.1

# Test Plan

- BareExpo
- ExpoGo

# Checklist

<!--
Please check the appropriate items below if they apply to your diff.
-->

- [ ] I added a `changelog.md` entry and rebuilt the package sources
according to [this short
guide](https://github.com/expo/expo/blob/main/CONTRIBUTING.md#-before-submitting)
- [ ] This diff will work correctly for `npx expo prebuild` & EAS Build
(eg: updated a module plugin).
- [ ] Conforms with the [Documentation Writing Style
Guide](https://github.com/expo/expo/blob/main/guides/Expo%20Documentation%20Writing%20Style%20Guide.md)
Eric-Tyrrell22 pushed a commit to Eric-Tyrrell22/expo that referenced this pull request Sep 5, 2025
# Why

callstack/react-native-slider#722 just got
released fixing support for 0.81, and we can now remove our patch
 
# How

Update @react-native-community/slider to 5.0.1

# Test Plan

- BareExpo
- ExpoGo

# Checklist

<!--
Please check the appropriate items below if they apply to your diff.
-->

- [ ] I added a `changelog.md` entry and rebuilt the package sources
according to [this short
guide](https://github.com/expo/expo/blob/main/CONTRIBUTING.md#-before-submitting)
- [ ] This diff will work correctly for `npx expo prebuild` & EAS Build
(eg: updated a module plugin).
- [ ] Conforms with the [Documentation Writing Style
Guide](https://github.com/expo/expo/blob/main/guides/Expo%20Documentation%20Writing%20Style%20Guide.md)
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.

4 participants