NativeScript / NativeScript/firebase
Expost remove listeners for firebase messaging
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- TypeScript
- Sterne
- 62
- Forks
- 53
- Ø Merge
- 8 T. 4 Std.
- Gemergte PRs (30 T.)
- 2
Beschreibung
When listening to onMessage, onNotificationTap there is no way to remove listeners and is leading to memory leak
// on message
firebase()
.messaging()
.onMessage(async (remoteMessage) => {
console.log({ remoteMessage });
});
// on notification tap
firebase()
.messaging()
.onNotificationTap((message) => {
console.log({ message });
});
there is no way to remove listeners as there is for firebase auth
firebase().auth().addAuthStateChangeListener
firebase().auth().removeAuthStateChangeListener
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginnen Sie damit, die Messaging-Implementierungen für onMessage und onNotificationTap zu lokalisieren, und untersuchen Sie anschließend, wie jeder Listener registriert und beibehalten wird. Die Arbeit ist abgeschlossen, wenn beide APIs eine Möglichkeit zum Entfernen ihrer Listener bieten, vergleichbar mit dem Beispiel für den Authentication-Listener, und das bestehende Messaging-Verhalten erhalten bleibt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- firebase, typescript
- Bereich
- mobile
- Issue-Typ
- Feature
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 42/100