search

johnpatrickmorgan / FlowStacks

FlowStacks allows you to hoist SwiftUI navigation and presentation state into a Coordinator
MIT License
832 stars 65 forks source link

Fix regression onDismiss property not used due to conflicts with another MR #16

Closed lionel-alves closed 2 years ago

lionel-alves commented 2 years ago

This MR: onDismiss passed to nil (https://github.com/johnpatrickmorgan/FlowStacks/commit/a9a3dcf6545b52c397e8b60931de01f92dd86c99) when fixing Adds a workaround for pre-iOS 14.5 presentation (https://github.com/johnpatrickmorgan/FlowStacks/issues/12)

johnpatrickmorgan commented 2 years ago

Ah, thanks for spotting that @lionel-alves! I must have introduced that error when rebasing.