search

jabardigitalservice / pikobar-flutter

Pikobar Flutter (Android) app.
https://play.google.com/store/apps/details?id=id.go.jabarprov.pikobar
MIT License
75 stars 50 forks source link

Change Analytics #519

Closed jaluowen closed 3 years ago

jaluowen commented 3 years ago
jaluowen commented 3 years ago

Review kang @aldiiii @gaganugraha33 @anandarh

jaluowen commented 3 years ago

Untuk screen ini mending pake log event yang biasa apa 'screen_view'? @anandarh

anandarh commented 3 years ago

@jaluowen Untuk penggunaan setCurrentScreen biarkan saja dulu, karena saya cek di packege firebase_analytics terbaru pun masih digunakan. Tunggu biar diupdate sama tim firebase flutternya langsung saja dulu.

jaluowen commented 3 years ago

ooh okay di close aja berarti ya ini PRnya? @anandarh

anandarh commented 3 years ago

@jaluowen Tapi kalo mau diubah, yang harus diubah adalah dibagian class AnalyticsHelper-nya. Biarkan nama methodnya tetap setCurrentScreen, tapi ganti dibagian sini:

await analytics.setCurrentScreen(
      screenName: screenName,
      screenClassOverride: screenClassOverride != null
          ? screenClassOverride
          : '${screenName.replaceAll(RegExp("\\s+"), "")}Screen',
    );

jadi

await analytics.logEvent(
      name: 'screen_view',
      parameters: <String, dynamic>{
        'screen_name': screenName,
        'screen_class': screenClassOverride != null
            ? screenClassOverride
            : '${screenName.replaceAll(RegExp("\\s+"), "")}Screen',
      },
    );