evollu / evollu/react-native-firebase-analytics
screen_view is reserved, but not in list of reserved names
- 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