NativeScript / NativeScript/firebase

Expost remove listeners for firebase messaging

Offen
#240 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

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

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Ö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

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.