[POS Settings] Feature flag and present settings view #16009
Conversation
|
|
jaclync
left a comment
jaclync
left a comment
There was a problem hiding this comment.
Works as expected 🚀 just a comment about deprecated NavigationView.
| struct PointOfSaleSettingsView: View { | ||
| @Environment(\.dismiss) private var dismiss | ||
| var body: some View { | ||
| NavigationView { |
There was a problem hiding this comment.
As NavigationView is deprecated, can we replace it with NavigationStack or NavigationSplitView?
There was a problem hiding this comment.
Yeah, I used the NavigationView just temporarily. I'm experimenting now with both Stack or SplitView to see what would make more sense here, as well as with a custom view like we do Itemlist/Cart depending how much flexibility the new APIs offer 👍
| Text("Settings") | ||
| .toolbar { | ||
| ToolbarItem(placement: .navigationBarTrailing) { | ||
| Button("Done") { |
There was a problem hiding this comment.
I assume these strings are just for the dummy view and will be replaced or localized later?
iamgabrielma
deleted the
task/WOOMOB-1021-feature-flag-and-settings-view-presentation
branch
Closes WOOMOB-1021
Description
This PR adds a feature flag for POS Settings i1, as well as a new button to the CTA entry point that will present a full screen settings dummy view.
Testing information
pointOfSaleSettingsi1totrueSettingsbuttonScreenshots
Screen.Recording.2025-08-14.at.11.21.03.mov
RELEASE-NOTES.txtif necessary.