feat(geofence): Adds geofence plugin

[amuramoto]

Closes #314

Supports all Cordova plugin functions except:

-onTransitionReceived()
-onNotificationClicked()

These functions require the base window.geofence object to be modified directly. See https://github.com/cowbell/cordova-plugin-geofence#listening-for-geofence-transitions

[ihadeed]

Why is there an edit in the camera preview plugin? is that there by accident?

[ihadeed]

Also, we can do something like this for the two methods you mentioned

// add this to the top of the document
declare var window: any;

static onTransitionReceived(): Observable<any> {

return new Observable<any>((observer) => {
window.geofence.onTransitionReceived = observer.next.bind(observer);
return () => window.geofence.onTransactionReceived = undefined
// undefined can be replaced with ()=>{} .. whichever works better
});

}

[amuramoto]

Oops must have cut the branch wrong. New commit reverts Camera Preview plugin for this branch to what's in master.

[amuramoto]

@ihadeed I tried your suggestion for onTransitionReceived, but it returns a ZoneAwarePromise rather than an observer. Any advice?

[ihadeed]

Are you using a Cordova decorator? You shouldn't use one

On Aug 17, 2016 5:59 PM, "Alex Muramoto" [email protected] wrote:

@ihadeed https://github.com/ihadeed I tried your suggestion for
onTransitionReceived, but it returns a ZoneAwarePromise rather than an
observer. Any advice?

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#442 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANJ8dNMb1mLsKd5SpSoIEmKslijaQ721ks5qg4RYgaJpZM4JlzmA
.

[amuramoto]

Removed the Cordova decorator. Function now returns the Observable, but the observer is never called.

[ihadeed]

Can you provide a code sample of how you're calling the plugin?

On Aug 19, 2016 12:38 AM, "Alex Muramoto" [email protected] wrote:

Removed the Cordova decorator. Function now returns the Observable, but
the observer is never called.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#442 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANJ8dCFxRQZhqsWjV7zBRQR1kMtAs73gks5qhTM-gaJpZM4JlzmA
.

[amuramoto]

Here's the plugin code:

static onTransitionReceived(): Observable<any> {

    return new Observable<any>((observer) => {
      window.geofence.onTransitionReceived = observer.next.bind(observer);
      return null;
      // undefined can be replaced with ()=>{} .. whichever works better
    });

  }

And here's the code calling it, which should update a data binding in the view, but does not appear to:

Geofence.onTransitionReceived().subscribe(()=>this.transited='TRANSITED');

[ihadeed]

Just fixed them. When we provide custom code for a wrapper we can't use @Cordova decorator as it overrides any code in our function. Thank @amuramoto