-
-
Notifications
You must be signed in to change notification settings - Fork 121
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
unable to resolve module @shopify/flash-list [Crash] #376
Comments
Got this too, just install |
upgrade below dependencies : degrade your react-native-action-sheet version to "0.9.3" , if that doesn't work, upgrade your version to latest version and then degrade to "0.9.3" again... this worked for me. 👍 |
I've fixed it by installing @shopify/flash-list like @goguda has mentioned it, thanks |
@rodrigodiasf1984 Are you able to reopen this issue? We should probably leave this open since I think this technically is a bug. Thank you! |
I don't understand why versions are shipped to npm which the code for it on repo doesn't exist. Last commit was even 3 months ago. Guys kindly install new versions with caution or better, stick to version 0.9.3 |
Hm, you do have a point, I did find this a little strange as well. |
So, just for peace of mind, I had a look at the code in version 0.9.6 shipped by npm and I don't see anything too out of the ordinary. I also used 0.9.4 looks like it may have been a mistake, pretty much everything in the dist folder gets deleted. 0.9.5 fixes the mistake by restoring what's in dist and 0.9.6 looks like it completes the migration from FlatList to shopify's FlashList (which is what introduced this error). So anyone who's already upgraded should be safe, for now. However, that having been said, thank you for drawing attention to this. I do still find this behaviour a bit alarming considering we're talking 3 revisions now where the code has not been updated on GitHub. There's nothing stopping the author from releasing a 0.9.7 version the same way with something malicious in it. The way everything gets deleted in 0.9.4 is also a bit strange. Not the most alarming thing, but could be testing the waters for something bigger perhaps? We're probably going to lock our package version and ween our app off of this package just to be safe. Everyone else should probably also take these measures, unless the author of the package @ammarahm-ed can shed some light on what's going on here. |
Thanks @goguda for spending time investigating this. 👏 I'd be up for making a fork, but I am not at a place where I can be the only maintainer |
This has been fixed in v0.9.7 release. v0.9.4 was an accident. I had added |
"react-native": "0.74.2",
"react-native-actions-sheet": "^0.9.6",
"react-native-gesture-handler": "^2.17.1",
Hi guys, just got this error , any idea how to solve it?
The text was updated successfully, but these errors were encountered: