Merge branch 'chore/nodejs-mobile-new-api' into 'main'

Change the API to remove nodejs listeners

See merge request bloodyhealth/drip!524

components/password-prompt.js

@@ -36,9 +36,9 @@ const PasswordPrompt = ({ enableShowApp }) => {
   }
 
   useEffect(() => {
-    nodejs.channel.addListener('check-pw', passHashToDb, this)
+    const listener = nodejs.channel.addListener('check-pw', passHashToDb, this)
 
-    return () => nodejs.channel.remove('check-pw', passHashToDb)
+    return () => listener.remove()
   }, [])
 
   const onDeleteDataConfirmation = async () => {

components/settings/common/confirm-with-password.js

@@ -24,10 +24,12 @@ const ConfirmWithPassword = ({ onSuccess, onCancel }) => {
   }
 
   useEffect(() => {
-    nodejs.channel.addListener('password-check', checkPassword, this)
-    return () => {
-      nodejs.channel.removeListener('password-check', checkPassword)
-    }
+    const listener = nodejs.channel.addListener(
+      'password-check',
+      checkPassword,
+      this
+    )
+    return () => listener.remove()
   }, [])
 
   const onIncorrectPassword = () => {