-
Notifications
You must be signed in to change notification settings - Fork 11
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
GEN-2462: Cleanup DK and NO #1439
base: main
Are you sure you want to change the base?
Conversation
@@ -86,24 +84,16 @@ public struct NotLoggedInView: View { | |||
} | |||
|
|||
@ViewBuilder | |||
var marketAndLanguage: some View { | |||
var language: some View { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we could actually name this one languageView for readability just to be extra clear that this is a view!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Makes sense! I'll fix it!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! 😄 waiting to merge until end of august
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lets remove all things that we don't use anymore or are overkill
@@ -17,7 +17,7 @@ extension AppDelegate { | |||
let optionalDictionary: [String: String?] = [ | |||
"memberId": memberId, | |||
"appVersion": Bundle.main.appVersion, | |||
"market": Localization.Locale.currentLocale.market.rawValue, | |||
"market": "se", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sladan-hedvig not sure how to handle this one
@@ -207,19 +207,7 @@ struct LoggedInNavigation: View { | |||
) | |||
.environmentObject(vm.profileNavigationVm) | |||
case .pickLanguage: | |||
PickLanguage { [weak profileNavigationVm = vm.profileNavigationVm, weak mainNavigationVm, weak vm] _ in |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sladan-hedvig should be fine right?
@@ -186,7 +183,7 @@ public class NotLoggedViewModel: ObservableObject { | |||
webUrl | |||
.appending("utm_source", value: "ios") | |||
.appending("utm_medium", value: "hedvig-app") | |||
.appending("utm_campaign", value: Localization.Locale.currentLocale.market.rawValue.lowercased()) | |||
.appending("utm_campaign", value: "se") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sladan-hedvig same, not sure how to handle
# Conflicts: # Projects/App/Sources/Journeys/LoggedInNavigation.swift # Projects/App/Sources/Journeys/LoginNavigation.swift # Projects/Market/Sources/LanguageAndMarketPickerView.swift # Projects/Market/Sources/PickLanguage.swift
# Conflicts: # Projects/hCore/Sources/Masking.swift
# Conflicts: # Projects/Claims/Sources/Views/ClaimDetailView.swift
# Conflicts: # Projects/App/Sources/AppDelegate+Unleash.swift # Projects/App/Sources/Journeys/LoginNavigation.swift # Projects/App/Sources/Screens/Impersonation/ImpersonationSettings.swift # Projects/Authentication/Sources/OTP/OTPEntryView.swift # Projects/Authentication/Sources/Service/OctopusImplementation/AuthenticationClientAuthLib.swift # Projects/Home/Sources/HomeState.swift # Projects/Market/Sources/LanguageAndMarketPickerView.swift # Projects/Market/Sources/MarketStore.swift # Projects/Market/Sources/PickLanguage.swift # Projects/MoveFlow/Sources/View/MovingFlowConfirm.swift # Projects/MoveFlow/Sources/View/MovingFlowHouseView.swift # Projects/MoveFlow/Sources/View/MovingFlowHousingType.swift # Projects/hCore/Sources/FeatureFlags/FeatureFlagOctopus.swift
# Conflicts: # Projects/App/Sources/Journeys/LoginNavigation.swift
# Conflicts: # Projects/Contracts/Sources/Models/ContractModels.swift # Projects/MoveFlow/Sources/View/MovingFlowConfirm.swift
[GEN-2462]
Checklist