react-native-fingerprint-scanner icon indicating copy to clipboard operation
react-native-fingerprint-scanner copied to clipboard

React Native 0.69.0: warn Package react-native-fingerprint-scanner has been ignored because it contains invalid configuration. Reason: "dependency.assets" is not allowed

Open anija opened this issue 2 years ago • 4 comments

Hi! 👋

Firstly, thanks for your work on this project! 🙂

Today I used patch-package to patch [email protected] for the project I'm working on.

When I upgraded React Native to latest version (0.69.0), i started to see this warning: warn Package react-native-fingerprint-scanner has been ignored because it contains invalid configuration. Reason: "dependency.assets" is not allowed

Here is the diff that solved my problem:

diff --git a/node_modules/react-native-fingerprint-scanner/react-native.config.js b/node_modules/react-native-fingerprint-scanner/react-native.config.js
index 5f794ab..c36ba55 100644
--- a/node_modules/react-native-fingerprint-scanner/react-native.config.js
+++ b/node_modules/react-native-fingerprint-scanner/react-native.config.js
@@ -4,8 +4,5 @@ module.exports = {
       ios: {},
       android: {},
     },
-    assets: [],
-    // hooks are considered anti-pattern, please avoid them
-    hooks: {},
   },
 };

This issue body was partially generated by patch-package.

anija avatar Jul 05 '22 10:07 anija

Oh this one is really important as it the CLI fails the module configuration completely until they release a new version @anija do you think you could post a full PR for this instead of just the patch?

mikehardy avatar Jul 05 '22 16:07 mikehardy

Oh this one is really important as it the CLI fails the module configuration completely until they release a new version @anija do you think you could post a full PR for this instead of just the patch?

I can't do enough test at this moment to feel comfortable in posting a PR :(

anija avatar Jul 05 '22 16:07 anija

That's okay in general I think - you post PR, and CI runs test, or then I have the ability to use github command line tool to easily pull the PR and run tests. No PR? No ability to use tools to do testing etc and integrating a patch for testing is manual process

mikehardy avatar Jul 05 '22 17:07 mikehardy

Flagging a PR like that (like "Hey, I'm posting this but have not had time to test, sorry, hopefully it works or is easier for others to test/verify this way"). It's a human process :-)

mikehardy avatar Jul 05 '22 17:07 mikehardy