Skip to content

1
1weather
Commit 308284dd by Demid Merzlyakov
Options

IOS-260: remove smart text from widget updated event for the radar widget.

parent fbd0c95e
Showing with 4 additions and 0 deletions
OneWeatherCore/OneWeatherCore/Managers/WidgetManager.swift
...@@ -142,10 +142,14 @@ public class WidgetManager { ...@@ -142,10 +142,14 @@ public class WidgetManager {
let name = WidgetOptions.name(for: option) let name = WidgetOptions.name(for: option)
var params: [AnalyticsParameter : String] = [.ANALYTICS_KEY_WIDGET_NAME : name] var params: [AnalyticsParameter : String] = [.ANALYTICS_KEY_WIDGET_NAME : name]
// Radar widget doesn't have smart text (IOS-260).
// TODO: this needs to be unified. Any widget not using smart text shouldn't have that parameter.
if option != .radarLarge {
if let updatedLocation = location { if let updatedLocation = location {
let smartText = self?.smartTextProvider.smartText(for: updatedLocation) ?? "" let smartText = self?.smartTextProvider.smartText(for: updatedLocation) ?? ""
params[.ANALYTICS_KEY_WIDGET_SMART_TEXT] = #"\#(smartText)"# params[.ANALYTICS_KEY_WIDGET_SMART_TEXT] = #"\#(smartText)"#
} }
}
AppAnalytics.shared.log(event: .ANALYTICS_WIDGET_UPDATED, AppAnalytics.shared.log(event: .ANALYTICS_WIDGET_UPDATED,
params: params) params: params)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment