From 1d7cd28f85d09d35805b59896809ca93aa436285 Mon Sep 17 00:00:00 2001 From: "Leonardo E. Dominguez" Date: Tue, 5 May 2020 17:52:23 -0400 Subject: [PATCH] fix: onTokenChange method causing app to crash (#3552) Removed deprecation. * fix: onTokenChange method causing app to crash * fix: use token instead of token with string ancestor Co-authored-by: Mike Diarmid --- packages/messaging/lib/index.js | 14 +------------- 1 file changed, 1 insertion(+), 13 deletions(-) diff --git a/packages/messaging/lib/index.js b/packages/messaging/lib/index.js index 804bca45ec..44c1260f4e 100644 --- a/packages/messaging/lib/index.js +++ b/packages/messaging/lib/index.js @@ -198,20 +198,8 @@ class FirebaseMessagingModule extends FirebaseModule { } const subscription = this.emitter.addListener('messaging_token_refresh', event => { - // TODO remove after v7.0.0, see: https://github.com/invertase/react-native-firebase/issues/2889 const { token } = event; - const tokenStringWithTokenAccessor = String(token); - Object.defineProperty(tokenStringWithTokenAccessor, 'token', { - enumerable: false, - get() { - // eslint-disable-next-line no-console - console.warn( - 'firebase.messaging().onTokenRefresh(event => event.token) is deprecated, use onTokenRefresh(token => token) or call getToken() instead', - ); - return token; - }, - }); - listener(tokenStringWithTokenAccessor); + listener(token); }); return () => subscription.remove(); }