nordnet / cordova-universal-links-plugin

[DEPRECATED] - Cordova plugin to support Universal/Deep Links for iOS/Android.
https://github.com/nordnet/cordova-universal-links-plugin/issues/160
MIT License
349 stars 530 forks source link

Cannot read property 'manifest' of undefined #133

Closed markovchainmontecarlo closed 6 years ago

markovchainmontecarlo commented 6 years ago

cordova-universal-links-plugin 1.2.1 "Universal Links Plugin"

cordova build Android Studio project detected (node:21980) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): TypeError: Cannot read property 'manifest' of undefined (node:21980) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

cordova platform version android Installed platforms: android 7.0.0 Available platforms: browser ~5.0.1 ios ~4.5.4 osx ~4.0.1 windows ~5.0.0 www ^3.12.0

cordova --version 8.0.0

npm --version 5.6.0

node --version v8.9.3

helenoalves commented 6 years ago

Hello Folks ! This issue is open yet !?

There is an workaround for this error ? I have exactly the same environment configuration as @jeremyspatrick !

@walteram Your fix worked to me too ! Thanks !

okaufmann commented 6 years ago

Is there any workaround till this get merged ?

helenoalves commented 6 years ago

@okaufmann look at the change of @walteram. It worked to me, only changing the path to store manifest.

okaufmann commented 6 years ago

Do I need to use his version of this plugin or move the AndroidManifest.xml file to the old path? I'm confused...

walteram commented 6 years ago

@okaufmann you can use the changed version of the plugin. since it's not merged in the nordnet repository, you must remove the plugin and install again using: cordova plugin add https://github.com/walteram/cordova-universal-links-plugin

hiepxanh commented 6 years ago

@walteram why dont you make a pull request? your code is working! Thank you! you save my life ❤ī¸

fonsecas72 commented 6 years ago

thanks @walteram :+1:

brnosouza commented 6 years ago

I changed a bit @walteram pull request to be backward compatible

rtm commented 6 years ago

Can this be merged at some point in the future, or an equivalent change made in the main branch, or is there some other plan? It's a little unsettling to have to be using a third party's fork.

lougreenwood commented 6 years ago

Any news? Looks like a new release just needs cutting?

skynixukraine commented 6 years ago

It still does not work nor iOS nor Android

monckey100 commented 6 years ago

This update needs to be pushed, I was adding the plugin and literally ran into it, seems like bad support that new users will just face this issue right off the bat and then have to resort to a third party branch.

HarelM commented 6 years ago

Same here, added the plug-in and bam, can't run my app. This is a showstopper guys...

nordnet-deprecation-bot commented 6 years ago

👋 Hi! Thank you for your interest in this repo.

đŸ˜ĸ We are not using nordnet/cordova-universal-links-plugin anymore, and we lack the manpower and the experience needed to maintain it. We are aware of the inconveniece that this may cause you. Feel free to use it as is, or create your own fork.

🔒 This will now be closed & locked.

ℹī¸ Please see #160 for more information.