Skip to content

Commit

Permalink
Merge pull request #43 from apachi1444/feature/fix-timber-log-event-w…
Browse files Browse the repository at this point in the history
…ith-properties

Add property numbering to logEvent method with properties
  • Loading branch information
aminekarimii committed Mar 6, 2024
2 parents 2812ec7 + 5b68f33 commit 16190b1
Showing 1 changed file with 11 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,17 @@ class TimberLocalClient : CoreAddon, EventsExtension, UserProfileExtension {
}

override fun logEvent(name: String, vararg properties: Param) {
Timber.tag(TAG).log(Log.INFO, "Event: $name -> props: [$properties]")
val logMessage = if (properties.isNotEmpty()) {
val propertyStrings = properties.mapIndexed { index, param ->
"${index + 1}. ${param.propertyName} : ${param.propertyValue}"
}.joinToString("\n")

"***** Event: $name -> props:\n$propertyStrings\n*****"
} else {
"***** Event: $name (No properties) *****"
}

Timber.tag(TAG).log(Log.INFO, logMessage)
}
override fun identify(userId: String) {
Timber.tag(TAG).log(Log.INFO, "User has been identified by: $userId")
Expand Down

0 comments on commit 16190b1

Please sign in to comment.