Closed gasolin closed 7 years ago
@ncloudioj could you help on this?
@gasolin Sure, will do.
Also, just a reminder that once we add those new columns to the database, the existing data records will not have values for them (i.e. the value will be null
). It might need some extra care when querying the database.
got it, thanks for the notice
related client code is landed into nightly https://bugzilla.mozilla.org/show_bug.cgi?id=1412164
As discussion on slack, this database change might be delayed due to the database scale up, but should be able to catch up before 57 hits the road
@gasolin This is now fixed.
Related to Bug 1412164 - add new column for onboarding telemetry (client side work), we also need add columns in server side.
for
firefox_onboarding_sessions_daily
table,tour_type
string column (valid string arenew
,update
, defaultupdate
)for
firefox_onboarding_events_daily
table,timestamp
column, (save event trigger time, same format as session_begin)tour_type
string column (valid string arenew
,update
, defaultupdate
)tour_source
string column (valid string aredefault
,watermark
, defaultdefault
, its the same exists infirefox_onboarding_sessions_daily
table)bubble_state
string column (not send this data now, will append in following client update) (valid string arebubble
,dot
,hide
, defaultbubble
)notification_state
string column (not send this data now, will append in following client update) (valid string areshow
,hide
,finish
, defaultshow
)And we will add
onboarding-icon-click
event infirefox_onboarding_events_daily
table, but I think it does not require column change