Merge pull request #1130 from WalletConnect/hotfix/listing-response-c…

…hanged [Notify] Listing decoding
WalletConnect · Sep 21, 2023 · 9f5719c · 9f5719c
2 parents 8019f7a + a277497
commit 9f5719c
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 7 deletions.
diff --git a/Example/WalletApp/BusinessLayer/ListingsSertice/Listings.swift b/Example/WalletApp/BusinessLayer/ListingsSertice/Listings.swift
@@ -13,8 +13,8 @@ struct Listing: Codable {
  }
  let id: String
  let name: String
- let description: String
- let homepage: String
- let image_url: ImageURL
+ let description: String?
+ let homepage: String?
+ let image_url: ImageURL?
  let dapp_url: String
 }
diff --git a/Example/WalletApp/PresentationLayer/Wallet/Notifications/Models/ListingViewModel.swift b/Example/WalletApp/PresentationLayer/Wallet/Notifications/Models/ListingViewModel.swift
@@ -9,19 +9,19 @@ struct ListingViewModel: Identifiable {
  }
 
  var imageUrl: URL? {
- return listing.image_url.md
+ return listing.image_url?.md
  }
 
  var title: String {
  return listing.name
  }
 
  var subtitle: String {
- return listing.description
+ return listing.description ?? ""
  }
 
  var appDomain: String? {
- let url = listing.homepage
+ let url = listing.dapp_url
  return URL(string: url)?.host
  }
 }
diff --git a/Example/WalletApp/PresentationLayer/Wallet/Notifications/NotificationsView.swift b/Example/WalletApp/PresentationLayer/Wallet/Notifications/NotificationsView.swift
@@ -42,7 +42,7 @@ struct NotificationsView: View {
  }
  }
  .task {
- try! await presenter.fetch()
+ try? await presenter.fetch()
  }
  }