Skip to content

Set native driver to false.#21

Open
johfarrell wants to merge 1 commit into
rafaelmotta:masterfrom
johfarrell:patch-1
Open

Set native driver to false.#21
johfarrell wants to merge 1 commit into
rafaelmotta:masterfrom
johfarrell:patch-1

Conversation

@johfarrell

Copy link
Copy Markdown

(WARN useNativeDriver was not specified. This is a required option and must be explicitly set to true or false) lead to random force close and apps reload in some cases.

(WARN `useNativeDriver` was not specified. This is a required option and must be explicitly set to `true` or `false`) lead to random force close and apps reload in some cases.
@jtalz

jtalz commented Oct 28, 2020

Copy link
Copy Markdown

I encountered this as well. It seems that it hasn't yet been changed in the repo. Thanks.

@MasIzzudin

Copy link
Copy Markdown

So, any update for this warning issue?

@johfarrell

Copy link
Copy Markdown
Author

The repo owner haven't merge my work yet. Still waiting for response

@denisvely

Copy link
Copy Markdown

@rafaelmotta Please merge this one.

@itamarbareket

Copy link
Copy Markdown

Hi just as a quick fix you can create this file as patches/react-native-progress-bar-animated+1.0.6.patch in your root repo dir and set it's value to

diff --git a/node_modules/react-native-progress-bar-animated/src/AnimatedProgressBar.js b/node_modules/react-native-progress-bar-animated/src/AnimatedProgressBar.js
index 8ba19be..4963b58 100644
--- a/node_modules/react-native-progress-bar-animated/src/AnimatedProgressBar.js
+++ b/node_modules/react-native-progress-bar-animated/src/AnimatedProgressBar.js
@@ -61,6 +61,7 @@ class ProgressBar extends React.Component {
       easing: Easing[this.props.barEasing],
       toValue: toValue > 0 ? toValue : 0,
       duration: this.props.barAnimationDuration,
+      useNativeDriver: false
     }).start();
   }

then run yarn patch-package to fix that.

You can also add to you package.json for a more automatic workflow:

 "scripts": {
+  "postinstall": "patch-package"
 }

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.

5 participants