Update package to use react-native-geolocation

[oakis]

Geolocation is being removed in RN 0.60 and is moved to https://github.com/react-native-community/react-native-geolocation

[JanithaR]

What do you mean by the title? Isn't this package a solution (alternative) for react-native-geolocation?

[oakis]

@JanithaR Reference: facebook/react-native#7495 (comment)

[JanithaR]

I know that and the recommended approach is to use react-native-geolocation-service instead of react-native-geolocation. I'm just still wondering what you meant in the question. Regards, Janitha

…

On Tue, May 14, 2019, 11:27 Anders Ekman ***@***.***> wrote: @JanithaR <https://github.com/JanithaR> Reference: facebook/react-native#7495 (comment) <facebook/react-native#7495 (comment)> — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#69?email_source=notifications&email_token=AA3O4AH45LV3AT3EIE6NG3LPVJID5A5CNFSM4HF7LMOKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVKLBZQ#issuecomment-492089574>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AA3O4AAC4NCQT6QMKFPAW63PVJID5ANCNFSM4HF7LMOA> .

[Agontuk]

react-native-geolocation is needed because this package uses RN's default implementation for iOS. Since it's moved into a separate repo we have to integrate that too. I can push a commit but can't test it cause I don't have any iOS device.

[canpoyrazoglu]

I've forked the repo (feel free to use if you need it before this repo is officially updated) at https://github.com/canpoyrazoglu/react-native-geolocation-service. (Though you'll need to add and link the dependency manually, see below.)

I can confirm that it works (at least, on iOS, couldn't test Android yet) after adding the geolocation package:

yarn add @react-native-community/geolocation

Linking it:
react-native link @react-native-community/geolocation

And referencing it in index.js (I've already done it in my fork):
import Geolocation from '@react-native-community/geolocation'

And also deleting the line (I've already done it in my fork):
let Geolocation = global.navigator.geolocation;
(as we'll be using the imported version as the same variable).

I'm on RN 0.60 RC2 and can confirm it works correctly on my device (iPhone X, iOS 12.3.1).

[Agontuk]

Sorry guys, I've been pretty busy with work lately so haven't pushed an update regarding this. I'll fix this tonight.

[Agontuk]

I've pushed a commit in master branch to fix this, please try and let me know if it works. I'll then release it in npm registry.

[c-schuette]

Hey, anythink new about adding it to the npm registry? And thanks for this helpful package :)

[Agontuk]

New version is already available in npm.