NativeScript / NativeScript/firebase
Expost remove listeners for firebase messaging
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
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- 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