Merge pull request #52 from 10up/fix/rework-authorisations

Fix/rework authorisations
10up · Jun 25, 2020 · 13a9674 · 13a9674
2 parents 078951e + 666fac8
commit 13a9674
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 16 deletions.
diff --git a/src/components/AppleMap.js b/src/components/AppleMap.js
@@ -54,21 +54,24 @@ class AppleMap {
 	}
 
 	static authenticateMap() {
-		apiFetch( { path: 'MapsBlockApple/v1/GetJWT/' } )
-			.then( ( token ) => {
-				mapkit.init( {
-					authorizationCallback( done ) {
-						done( token );
-					},
+		function getJWTToken( resolveCallback ) {
+			apiFetch( { path: 'MapsBlockApple/v1/GetJWT/' } )
+				.then( resolveCallback )
+				.catch( ( error ) => {
+					dispatch( 'core/notices' ).createErrorNotice(
+						error.message,
+						{
+							isDismissible: true,
+							type: 'snackbar',
+						}
+					);
+					mapkit.dispatchEvent( new Event( 'error' ) );
 				} );
-			} )
-			.catch( ( error ) => {
-				dispatch( 'core/notices' ).createErrorNotice( error.message, {
-					isDismissible: true,
-					type: 'snackbar',
-				} );
-				mapkit.dispatchEvent( new Event( 'error' ) );
-			} );
+		}
+
+		mapkit.init( {
+			authorizationCallback: getJWTToken,
+		} );
 	}
 }
 

diff --git a/src/edit.js b/src/edit.js
@@ -52,8 +52,8 @@ export default function MapsBlockAppleEdit( props ) {
 					break;
 				case 'Refreshed':
 					setIsLoading( false );
-					setAuthenticated( false );
-					mapkit.authenticated = false;
+					setAuthenticated( true );
+					mapkit.authenticated = true;
 					break;
 				default:
 					setIsLoading( false );