Add content reader property to link_read event
This commit is contained in:
@ -17,12 +17,11 @@ import Views
|
||||
if let item = item {
|
||||
pdfItem = PDFItem.make(item: item)
|
||||
self.item = item
|
||||
trackReadEvent(reader: item.isPDF ? "PDF" : "WEB")
|
||||
}
|
||||
|
||||
trackReadEvent()
|
||||
}
|
||||
|
||||
private func trackReadEvent() {
|
||||
private func trackReadEvent(reader: String) {
|
||||
guard let itemID = item?.unwrappedID ?? pdfItem?.itemID else { return }
|
||||
guard let slug = item?.unwrappedSlug ?? pdfItem?.slug else { return }
|
||||
guard let originalArticleURL = item?.unwrappedPageURLString ?? pdfItem?.downloadURL else { return }
|
||||
@ -31,6 +30,7 @@ import Views
|
||||
.linkRead(
|
||||
linkID: itemID,
|
||||
slug: slug,
|
||||
reader: reader,
|
||||
originalArticleURL: originalArticleURL
|
||||
)
|
||||
)
|
||||
|
||||
@ -25,6 +25,7 @@ import Views
|
||||
.linkRead(
|
||||
linkID: item.unwrappedID,
|
||||
slug: item.unwrappedSlug,
|
||||
reader: "WEB",
|
||||
originalArticleURL: item.unwrappedPageURLString
|
||||
)
|
||||
)
|
||||
|
||||
@ -1,7 +1,7 @@
|
||||
import Foundation
|
||||
|
||||
public enum TrackableEvent {
|
||||
case linkRead(linkID: String, slug: String, originalArticleURL: String)
|
||||
case linkRead(linkID: String, slug: String, reader: String, originalArticleURL: String)
|
||||
case debugMessage(message: String)
|
||||
case backgroundFetch(jobStatus: BackgroundFetchJobStatus, itemCount: Int, secondsElapsed: Int)
|
||||
case audioSessionStart(linkID: String)
|
||||
@ -33,10 +33,11 @@ public extension TrackableEvent {
|
||||
|
||||
var properties: [String: String]? {
|
||||
switch self {
|
||||
case let .linkRead(linkID: linkID, slug: slug, originalArticleURL: originalArticleURL):
|
||||
case let .linkRead(linkID: linkID, slug: slug, reader: reader, originalArticleURL: originalArticleURL):
|
||||
return [
|
||||
"link": linkID,
|
||||
"slug": slug,
|
||||
"reader": reader,
|
||||
"url": originalArticleURL
|
||||
]
|
||||
case let .debugMessage(message: message):
|
||||
|
||||
Reference in New Issue
Block a user