Merge pull request #29 from GeoTecINIT/migrate-rxjs

Migrate to RxJS 7

Author: agonper

README.md

@@ -226,13 +226,12 @@ async function printLocationUpdates(): Promise<Subscription> {
           maxAge: 60000, // And filter-out old locations
       });
 
-    return stream.subscribe(
-        (location) => {
-            console.log(`New location acquired!: ${JSON.stringify(location)}`);
-        },
-        (error) =>
+    return stream.subscribe({
+        next: (location) =>
+            console.log(`New location acquired!: ${JSON.stringify(location)}`),
+        error: (error) =>
             console.error(`Location updates could not be acquired: ${error}`)
-    );
+    });
 }
 ```
 

demo/app/home/home-page.ts

@@ -11,10 +11,7 @@ import { HomeViewModel } from "./home-view-model";
 import { contextApis } from "nativescript-context-apis";
 import { Resolution } from "nativescript-context-apis/activity-recognition";
 
-import {
-    GeolocationProvider,
-    Geolocation,
-} from "nativescript-context-apis/geolocation";
+import { GeolocationProvider } from "nativescript-context-apis/geolocation";
 import { of, Subscription } from "rxjs";
 
 const activityRecognizers = [Resolution.LOW, Resolution.MEDIUM];
@@ -83,15 +80,14 @@ async function printLocationUpdates(): Promise<Subscription> {
               timeout: 5000,
               maxAge: 60000,
           })
-        : of<Geolocation>(null);
+        : of(null);
 
-    return stream.subscribe(
-        (location) => {
-            console.log(`New location acquired!: ${JSON.stringify(location)}`);
-        },
-        (error) =>
+    return stream.subscribe({
+        next: (location) =>
+            console.log(`New location acquired!: ${JSON.stringify(location)}`),
+        error: (error) =>
             console.error(`Location updates could not be acquired: ${error}`)
-    );
+    });
 }
 
 export function listenToActivityChanges(addListener = false) {

demo/package-lock.json

@@ -10,7 +10,7 @@
     "@nativescript/theme": "~2.3.0",
     "@nativescript/unit-test-runner": "^1.0.2",
     "nativescript-context-apis": "file:../src",
-    "rxjs": "^6.5.5"
+    "rxjs": "^7.5.5"
   },
   "devDependencies": {
     "@nativescript/android": "7.0.1",

demo/package.json

@@ -63,7 +63,7 @@
 		"@nativescript/geolocation": "^8.0.2",
 		"@triniwiz/nativescript-couchbase": "1.2.5",
 		"nativescript-permissions": "1.3.11",
-		"rxjs": "^6.5.5"
+		"rxjs": "^7.5.5"
 	},
 	"bootstrapper": "nativescript-plugin-seed"
 }