Skip to content

Commit

Permalink
Merge pull request #1738 from streamich/fix-1646
Browse files Browse the repository at this point in the history
fix: #1646
  • Loading branch information
xobotyi authored Jan 8, 2021
2 parents 12894dc + ebc7094 commit 5d5bcc8
Showing 1 changed file with 15 additions and 2 deletions.
17 changes: 15 additions & 2 deletions src/useGeolocation.ts
Original file line number Diff line number Diff line change
@@ -1,5 +1,18 @@
import { useEffect, useState } from 'react';

/**
* @desc Made compatible with {GeolocationPositionError} and {PositionError} cause
* PositionError been renamed to GeolocationPositionError in typescript 4.1.x and making
* own compatible interface is most easiest way to avoid errors.
*/
export interface IGeolocationPositionError {
readonly code: number;
readonly message: string;
readonly PERMISSION_DENIED: number;
readonly POSITION_UNAVAILABLE: number;
readonly TIMEOUT: number;
}

export interface GeoLocationSensorState {
loading: boolean;
accuracy: number | null;
Expand All @@ -10,7 +23,7 @@ export interface GeoLocationSensorState {
longitude: number | null;
speed: number | null;
timestamp: number | null;
error?: Error | PositionError;
error?: Error | IGeolocationPositionError;
}

const useGeolocation = (options?: PositionOptions): GeoLocationSensorState => {
Expand Down Expand Up @@ -43,7 +56,7 @@ const useGeolocation = (options?: PositionOptions): GeoLocationSensorState => {
});
}
};
const onEventError = (error: PositionError) =>
const onEventError = (error: IGeolocationPositionError) =>
mounted && setState((oldState) => ({ ...oldState, loading: false, error }));

useEffect(() => {
Expand Down

0 comments on commit 5d5bcc8

Please sign in to comment.