n4kz/react-native-material-dropdown

Not working with RN v62

ganesh-papola opened this issue · 15 comments

ganesh-papola commented

react-native-material-dropdown has dependency on react-native-material-textfield

latest react native has removed proptypes which is causing the issue on react-native-material-textfield please remove proptypes or fix this

Atom1er commented

Same issue here

ganesh-papola commented

@Atom1er Thanks for the reference.
But this is temporary solution . I don't want to edit node modules packages as I have multiple developers working project and yarn command is often. which removes everything....

  • 👍2
BenHurMartins commented

Same issue here, the workaround should be seem as temporary, when you have more than 1 developer working in the project.

jacktiong92 commented

Same issue here

SoulKa commented

the app doesn't even start when using the newest expo SDK with this package. Important breaking changes

rdgomt commented

same issue here

AtmasApps commented

Now I'm broken too. I followed the instructions on this bug patch which led me too some PRs that successfully implemented the handful of renames to Animated...etc that are required. Hoping @n4kz can merge and release, but looks like maybe he's busy?

Would be great to have textfield updated, released, and then this one released with the new commensurate dependency!

n4kz/react-native-material-textfield#254

OscarYuen commented

I made a patch to make react-native-material-dropdown work in RN 0.62.

Requirement:

  1. Install patch-package
  2. Run yarn add patch-package postinstall-postinstall or npm i patch-package
  3. Your react-native-material-textfield dependency must be installed under react-native-material-dropdown node_modules. If not, please use the below patch or the patch from this thread instead(n4kz/react-native-material-textfield#249)

Instruction:

  1. Create "patches" directory under your project's root
  2. Copy this patch to patches/react-native-material-dropdown++react-native-material-textfield+0.12.0.patch
  3. Run yarn patch-package or npx patch-package
  4. Fix is already applied. Add the following to package.json to make sure the fix is automatically applied next time:
 "scripts": {
    "postinstall": "patch-package"
 }

https://gist.github.com/OscarYuen/21f2f8d5c133caef7d31475cfec2d5b0#file-react-native-material-dropdown-react-native-material-textfield-0-12-0-patch

  • 👍1
  • ❤️2
bguaitanele commented

This patch didn't work here. The patch assumes react-native-material-textfield is in node_module inside react-native-material-dropdown. In my project, although the textfield is used only in dropdown, it was installed in my node_modules root by yarn decision. So, when I try to run the patch, the following error occurs:

Error: Patch file found for package react-native-material-textfield which is not present at node_modules/react-native-material-dropdown/node_modules/react-native-material-textfield

The error occur after upgrade my expo from 37 to 38, which upgraded RN to 0.62

  • 👍2
jehartzog commented

@bguaitanele I found the same issue you did, took a couple minutes to fix. gist here.

  • 👍3
jehartzog commented

While my first revision fix the hard crash issues, I was still getting some propType warnings and other console spam. I revised the above gist to clear those, and cleaned up a few more:
react-native-material-buttons
react-native-material-dropdown

OscarYuen commented

This patch didn't work here. The patch assumes react-native-material-textfield is in node_module inside react-native-material-dropdown. In my project, although the textfield is used only in dropdown, it was installed in my node_modules root by yarn decision. So, when I try to run the patch, the following error occurs:

Error: Patch file found for package react-native-material-textfield which is not present at node_modules/react-native-material-dropdown/node_modules/react-native-material-textfield

The error occur after upgrade my expo from 37 to 38, which upgraded RN to 0.62

For this kind of structure, I suggest to use the patch from react-native-material-textfield directly(n4kz/react-native-material-textfield#249).
Some people there made the patch for textfield directly.

bguaitanele commented

@bguaitanele I found the same issue you did, took a couple minutes to fix. gist here.

I was doing the same fix here, lol. You're faster. Anyway, for those who tried the first patch, before the second, maybe could have an error because the first fix. So, just delete your node_modules, run yarn ou npm install again, then run the second fix. Worked for me!