Skip to content
This repository has been archived by the owner on Oct 1, 2018. It is now read-only.

Cannot read property 'manifest' of undefined #133

Closed
markovchainmontecarlo opened this issue Dec 28, 2017 · 14 comments
Closed

Cannot read property 'manifest' of undefined #133

markovchainmontecarlo opened this issue Dec 28, 2017 · 14 comments

Comments

@markovchainmontecarlo
Copy link

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

walteram added a commit to walteram/cordova-universal-links-plugin that referenced this issue Dec 29, 2017
walteram added a commit to walteram/cordova-universal-links-plugin that referenced this issue Dec 29, 2017
Update manifestWriter.js to fix nordnet#133 in Cordova 8.
@helenoalves
Copy link

helenoalves commented Jan 24, 2018

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
Copy link

Is there any workaround till this get merged ?

@helenoalves
Copy link

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

@okaufmann
Copy link

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

@walteram
Copy link

@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

balat pushed a commit to besport/cordova-universal-links-plugin that referenced this issue Mar 5, 2018
balat added a commit to besport/cordova-universal-links-plugin that referenced this issue Mar 5, 2018
Update manifestWriter.js to fix nordnet#133 in Cordova 8.
@hiepxanh
Copy link

hiepxanh commented May 7, 2018

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

@fonsecas72
Copy link

thanks @walteram 👍

@brnosouza
Copy link

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

@rtm
Copy link

rtm commented Jun 11, 2018

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.

tiltec added a commit to karrot-dev/karrot-frontend that referenced this issue Jun 15, 2018
cordova-universal-links-plugin isn't compatible with cordova 7
nordnet/cordova-universal-links-plugin#133
tiltec added a commit to karrot-dev/karrot-frontend that referenced this issue Jun 15, 2018
cordova-universal-links-plugin isn't compatible with cordova 7

nordnet/cordova-universal-links-plugin#133
tiltec added a commit to karrot-dev/karrot-frontend that referenced this issue Jun 15, 2018
* WIP upgrade

* use stubs from test-utils

* delay infinite scroll retry if it can't load more

* fix wrong usage of toast message

* dep update

* upgrade vue2-leaflet

* downgrade clone-deep

* upgrade android build image

* upgrade cordova-android

* update android instructions

* use better gradle caching

* restore cordova-android 6.4

cordova-universal-links-plugin isn't compatible with cordova 7

nordnet/cordova-universal-links-plugin#133
@lougreenwood
Copy link

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

@skynixukraine
Copy link

It still does not work nor iOS nor Android

@monckey100
Copy link

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
Copy link

HarelM commented Sep 17, 2018

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

@nordnet-deprecation-bot
Copy link
Contributor

👋 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.

@nordnet nordnet locked and limited conversation to collaborators Sep 19, 2018
tdelmas pushed a commit to tdelmas/cordova-universal-links-plugin that referenced this issue Oct 2, 2018
alexstanbury added a commit to alexstanbury/cordova-universal-links-plugin that referenced this issue Nov 7, 2018
MulderWH added a commit to MulderWH/cordova-universal-links-plugin that referenced this issue Jan 25, 2019
Update manifestWriter.js to fix nordnet#133 in Cordova 8.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests