NativeScript / NativeScript/firebase

Expost remove listeners for firebase messaging

Ouverte
#240 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Langage dominant
TypeScript
Étoiles
62
Forks
53
Merge moyen
8 j 4 h
PR mergées (30 j)
2

Description

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

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez par localiser les implémentations de messaging pour onMessage et onNotificationTap, puis examinez comment chaque listener est enregistré et conservé. Le travail est terminé lorsque les deux APIs fournissent un moyen de supprimer leurs listeners, comparable à l’exemple du listener d’authentification, et que le comportement existant de messaging reste intact.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
firebase, typescript
Domaine
mobile
Type d'issue
Fonctionnalité
Difficulté
3/5
Temps estimé
1-2 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
42/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.