Skip to content

[WCiOS17] Update PaymentMethodsView to not use deprecated NavigationLink init #16196

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.

Sign up for GitHub

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

Merged
merged 4 commits into from
Oct 17, 2025
Merged

[WCiOS17] Update PaymentMethodsView to not use deprecated NavigationLink init #16196

Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
379d7c2
remove usage of NavigationLink.isActive init
iamgabrielma Oct 2, 2025
33a996d
fix previews
iamgabrielma Oct 2, 2025
84d2e6a
Use NavigationStack in previews
iamgabrielma Oct 3, 2025
f0b6d85
Merge branch 'trunk' into task/wcios17-payment-method
iamgabrielma Oct 17, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
54 changes: 20 additions & 34 deletions WooCommerce/Classes/ViewRelated/Orders/Payment Methods/PaymentMethodsView.swift
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -101,17 +101,6 @@ struct PaymentMethodsView: View {
) {
AttributedText(viewModel.learnMoreViewModel.learnMoreAttributedString)
}.padding(.horizontal)

NavigationLink(isActive: $showingCashAlert) {
CashPaymentTenderView(viewModel: CashPaymentTenderViewModel(total: viewModel.total, formattedTotal: viewModel.formattedTotal) { info in
Task { @MainActor in
await viewModel.markOrderAsPaidByCash(with: info)
dismiss()
}
})
} label: {
EmptyView()
}.hidden()
}

// Pushes content to the top
Expand Down Expand Up @@ -145,6 +134,15 @@ struct PaymentMethodsView: View {
}
.background(FullScreenCoverClearBackgroundView())
}
.navigationDestination(isPresented: $showingCashAlert) {
CashPaymentTenderView(viewModel: CashPaymentTenderViewModel(total: viewModel.total,
formattedTotal: viewModel.formattedTotal) { info in
Task { @MainActor in
await viewModel.markOrderAsPaidByCash(with: info)
dismiss()
}
})
}
.onAppear {
guard rootViewController != nil else {
return viewModel.logNoRootViewControllerError()
Expand Down Expand Up @@ -252,28 +250,16 @@ extension PaymentMethodsView {
}
}

// MARK: Previews
struct PaymentMethodsView_Preview: PreviewProvider {
static var previews: some View {
NavigationView {
PaymentMethodsView(viewModel: .init(total: "15.99", formattedTotal: "$15.99", flow: .orderPayment, channel: .storeManagement))
.navigationBarTitleDisplayMode(.inline)
}
.environment(\.colorScheme, .light)
.previewDisplayName("Light")

NavigationView {
PaymentMethodsView(viewModel: .init(total: "15.99", formattedTotal: "$15.99", flow: .orderPayment, channel: .storeManagement))
.navigationBarTitleDisplayMode(.inline)
}
.environment(\.colorScheme, .dark)
.previewDisplayName("Dark")

NavigationView {
PaymentMethodsView(viewModel: .init(total: "15.99", formattedTotal: "$15.99", flow: .orderPayment, channel: .storeManagement))
.navigationBarTitleDisplayMode(.inline)
}
.environment(\.sizeCategory, .accessibilityExtraExtraLarge)
.previewDisplayName("Accessibility")
#Preview {
@Previewable @State var rootViewController = UIViewController()
NavigationStack {
PaymentMethodsView(
rootViewController: rootViewController,
viewModel: PaymentMethodsViewModel(total: "15.99",
formattedTotal: "$15.99",
flow: .orderPayment,
channel: .storeManagement)
)
.navigationBarTitleDisplayMode(.inline)
}
}