Closed aramando closed 6 years ago
When can this ticket be merged to master?
Yup, please merge this and release to npm :)
@nikDemyankov please merge this pull request. It's needed can't run my ios app without it!
thanks
Did this get merged in? Not sure how to tell.
No, it hasn't been merged, I don't know why; it seems nobody is updating this plugin at the moment.
I would suggest just referencing this commit with the fix as your plugin source instead of the nordnet master. If you're uncomfortable relying on my repository you can always fork it and maintain your own source.
cordova plugin add --save https://github.com/aramando/cordova-universal-links-plugin#35b3ed7e9a0310b12f1ac92a5159b21ce50eee57
I needed to add a .git
to the Github URL, to make it work. Also --save
seems to be the default now.
cordova plugin add https://github.com/aramando/cordova-universal-links-plugin.git#35b3ed7e9a0310b12f1ac92a5159b21ce50eee57
Other than that, I can confirm, that this change works as expected. Thanks for the help.
Is there any plans on merging this PR?
đ 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.
Fix for issue #110