Skip to content

Commit

Permalink
Update survey builder OS version (#984)
Browse files Browse the repository at this point in the history
Required:

Task/Issue URL: https://app.asana.com/0/414235014887631/1208279548359774/f
iOS PR: duckduckgo/iOS#3348
macOS PR: duckduckgo/macos-browser#3247
What kind of version bump will this require?: Patch

Description:

This PR updates the survey builder to not include the major.minor.patch OS version, but instead only the major version.
  • Loading branch information
samsymons authored Sep 12, 2024
1 parent 0021e32 commit e304d39
Showing 1 changed file with 4 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,10 @@ public struct DefaultRemoteMessagingSurveyURLBuilder: RemoteMessagingSurveyActio
queryItems.append(URLQueryItem(name: parameter.rawValue, value: variant))
}
case .osVersion:
queryItems.append(URLQueryItem(name: parameter.rawValue, value: AppVersion.shared.osVersion))
let os = ProcessInfo().operatingSystemVersion
let version = "\(os.majorVersion)"

queryItems.append(URLQueryItem(name: parameter.rawValue, value: version))
case .appVersion:
queryItems.append(URLQueryItem(name: parameter.rawValue, value: AppVersion.shared.versionAndBuildNumber))
case .hardwareModel:
Expand Down

0 comments on commit e304d39

Please sign in to comment.