evollu / evollu/react-native-firebase-analytics

screen_view is reserved, but not in list of reserved names

Offen
#55 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
Java
Sterne
206
Forks
52
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

This package includes a list of reserved event names which doesn't include "screen_view". So tracking events manually using `logEvent("screen_view")` will actually fail. I encountered this since `setCurrentScreen(...)` doesn't actually track anything and the automatic tracking doesn't work (likely because RN projects usually don't use native navigation components.

Fix is simply to add 1 line in `lib/modules/analytics/index.js` in the `const ReservedEventNames` array: `'screen_view'`

Thanks

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Öffne lib/modules/analytics/index.js und finde das const ReservedEventNames-Array. Füge screen_view zu den reservierten Namen hinzu und überprüfe anschließend, dass das manuelle Protokollieren dieses Events konsistent mit den anderen reservierten Event-Namen behandelt wird. Das Issue hat keine benannte Testdatei; abgeschlossen ist die aktualisierte Array und die korrigierte Event-Behandlung.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
firebase, javascript, react-native
Bereich
analytics, mobile-dev
Issue-Typ
Bug
Schwierigkeit
1/5
Geschätzter Aufwand
Unter einer Stunde
Aktivitätsstatus
Veraltet
Klarheit
Klar beschrieben
Anfängerfreundlichkeit
48/100

Neue Issues direkt in Ihr Postfach

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