inaturalist / iNaturalistAndroid

Android app for iNaturalist.org
https://market.android.com/details?id=org.inaturalist.android
MIT License
160 stars 57 forks source link

Incorporating Amplitude Events into Android app #305

Closed joellebel closed 6 years ago

joellebel commented 7 years ago

Here is a list of all Amplitude events on iOS

https://github.com/inaturalist/INaturalistIOS/blob/master/INaturalistIOS/Analytics.m#L123

I went through the events and they seem pretty parallel to Android.

Just a clarification on Settings... @"Setting Enabled"; tracks when any setting with only on/off options is enabled so in Android's case it would only be for the Automatic Upload setting.

Also, the events that include the word "Navigate" are for when viewing the actual screen, not when a particular button is pressed. For example, @"Navigate - Onboarding - Learn"; is the event triggered when the user is viewing the Onboarding - Learn slide.

Let us know if you have any questions!

budowski commented 7 years ago

Implemented in build 200 - some notes:

  1. "Obs - Add Identification" - added a new "Via" param value of "View Obs Add" (when adding a new identification and not just agreeing on an existing one).
  2. "Navigate - Observations - Details" - didn't implement the "Via" param value of "Updates" (since the app doesn't have an updates tab yet).
  3. "Navigate - Observations - Details" - added a new "Via" param of "Identifications Tab".
  4. "Explore - Add Comment" / "Explore - Add Identification" - not implemented in app.
  5. "Sync Observation" - no support for "Manual Single Upload" mode (not implemented in app).
  6. "Create Observation" - not implemented because it's identical to "New Obs - Save New Observation".
  7. "New Obs - Confirm Photos" / "New Obs - Retake Photos" - we have no knowledge when user confirms/retakes photos (it's out of the app's control).
  8. "Obs - Add Photo" - "Source" param is "Gallery" in case photos were imported (and not taken via app).
  9. "Obs - Location Changed" - as in the iOS app - event only sent when the user edits the location using the location editor screen (and not when using "Get current location" or when the app detects current location automatically).
  10. "Obs - Taxon Changed" - implemented, but we have no "No Taxon" value for "New Value" parameter (no way to completely delete a species guess for an existing obs, just change it).
  11. "Obs - Share Finished" - unfortunately in Android, we don't have a generic way of knowing if a sharing activity was sucessful or not (we always get a RESULT_CANCELED - see answer here).
  12. "Background Fetch Failed" - could not see if it's possible to implement it in the Android app (@alexshepard ?)
  13. Question: News events - just for user news or project news? @kueda / @alexshepard / @joellebel ?
  14. "News - Share Started" / "News - Share Cancelled" / "News - Share Finished" - no sharing capabilities for news in the app.
kueda commented 7 years ago

"Navigate - Observations - Details" - didn't implement the "Via" param value of "Updates" (since the app doesn't have an updates tab yet).

It does now, so would you add that?

"Explore - Add Comment" / "Explore - Add Identification" - not implemented in app.

Maybe I don't understand this event, but you can add comments and IDs from Explore.

Question: News events - just for user news or project news? @kueda / @alexshepard / @joellebel ?

I think these only apply to the content in the "News" tab of the "Activity" tab. You definitely can't share stuff in the "My Content" tab in iOS.

budowski commented 7 years ago

@kueda - "Explore - Add Comment/ID" - how can we add a comment/ID directly from the explore screen? We can do this indirectly (open an existing obs from the explore screen, and then add a comment/ID) "Via" param value of "Updates" - will add now

kueda commented 7 years ago

Re: Explore - Add Identification, I see what you're saying. @alexshepard, in iOS how is Obs - Add Identification different from Explore - Add Identification?

budowski commented 7 years ago

@kueda - build 206 will have the via value of "Updates"

joellebel commented 6 years ago

@budowski Can I close this?

budowski commented 6 years ago

Yep :-)