Migrate to FCM Cloud Messaging and dagger-android

the-blue-alliance / the-blue-alliance-android

An Android app for accessing information about the FIRST Robotics Competition.

MIT License

76 stars 34 forks source link

Migrate to FCM Cloud Messaging and dagger-android #955

Closed phil-lopreiato closed 2 years ago

phil-lopreiato commented 2 years ago

Okay this became a whole huge thing.

Basically, since Android 12 is stricter about PendingIntent flags, the way we used to pull GCM tokens and register them broke, since the library used PendingIntent internally without the flags.

So, this will:

move registration to the new, shiny WorkerManager APIs, which replaces JobIntentService
migrates all the DI to dagger-android, because that was actually the easiest way to get the worker to be injected
use FCM APIs locally, not GCM

phil-lopreiato commented 2 years ago

So on the hilt stuff, it sort of spiraled. Old IntentService stuff had to be migrated to WorkManager, which really can only be injected by hilt, which required redoing the whole app's DI...