-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Lint Warnings - Resolve MissingFirebaseInstanceTokenRefresh #18185
Description
As part of the #18161 task, this MissingFirebaseInstanceTokenRefresh got suppressed and moved within the lint.xml file (instead of it being within the baseline.xml file and multiplied).
This was done because in order to make this warning app, product flavor and build type agnostic as it was otherwise polluting the baseline with 6+ duplicate warnings, per app (WordPress and Jetpack), per product flavor (Jalapeno, Wasabi and Vanilla) and build type (Debug and Release).
However, this warning should be fixed instead and that's what this issue is about, to try and resolve these warnings instead of suppressing it (expand to see details).
Issue Details
<issue
id="MissingFirebaseInstanceTokenRefresh"
message="Apps that use Firebase Cloud Messaging should implement `onNewToken()` in order to observe token changes"
errorLine1="abstract class Hilt_GCMMessageService extends FirebaseMessagingService implements GeneratedComponentManagerHolder {"
errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~">
<location
file="build/generated/source/kapt/**/org/wordpress/android/push/Hilt_GCMMessageService.java"
line="16"
column="16"/>
</issue>Message: Apps that use Firebase Cloud Messaging should implement 'onNewToken()' in order to observe token changes"