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

Ionic Native 3.x #199

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Ionic Native 3.x #199

wants to merge 4 commits into from

Conversation

vknabel
Copy link

@vknabel vknabel commented Apr 19, 2017

Hi. We want to use Ionic Native 3.x, which is currently not possible with @ionic/[email protected].

As #190 has been opened a while ago, I wanted to help with this.
As all native plugins have to be instantiated (ideally through injection), I added them to the DIContainer class.

When trying npm run build it failed on master.

package.json Outdated
"superagent": "1.7.2",
"@ionic/db": "^0.1.0"
},
"peerDependencies": {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you upgrade the dependencies to native 3.6.1?

@vknabel
Copy link
Author

vknabel commented May 15, 2017

How shall I resolve the merge conflicts? Shall I retain window.plugins.googleplus.logout etc. or shall I rely on this.googlePlus.logout?

@janpio
Copy link

janpio commented May 17, 2017

Without understanding anything else what you did here: window.plugins.googleplus.logout would be using the Cordova plugin directly, this.googlePlus.logout would mean communicating with it via the Ionic Native plugin, correct? If so, the second one should be preferred.

@vknabel
Copy link
Author

vknabel commented May 18, 2017

@piotrowski Yes I know. I just don't know wether those changes were just a temporary and hacky workaround or wether they were meant to stay.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants