Skip to content

Add new alert API#73

Merged
mikhailChelbaev merged 4 commits intodevfrom
alert-with-item
Mar 13, 2025
Merged

Add new alert API#73
mikhailChelbaev merged 4 commits intodevfrom
alert-with-item

Conversation

@mikhailChelbaev
Copy link
Collaborator

@mikhailChelbaev mikhailChelbaev commented Mar 10, 2025

  • added a new method to present an alert with item:
func suAlert<Item>(
    item: Binding<Item?>,
    model: @escaping (Item) -> AlertVM,
    primaryAction: ((Item) -> Void)? = nil,
    secondaryAction: ((Item) -> Void)? = nil,
    onDismiss: (() -> Void)? = nil
) -> some View where Item : Identifiable
  • changed methods to present modals with items:
    param model: ModalVM, replaced with model: @escaping (Item) -> ModalVM
  • fixed bug where modals and alerts were not displayed at app launch when isPresented = true

@mikhailChelbaev mikhailChelbaev self-assigned this Mar 10, 2025
@mikhailChelbaev mikhailChelbaev merged commit eb7a1c5 into dev Mar 13, 2025
1 check passed
@mikhailChelbaev mikhailChelbaev deleted the alert-with-item branch March 13, 2025 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants