Use NonEmptyString

radixdlt · Aug 16, 2023 · b81019f · b81019f
1 parent 57b47d2
commit b81019f
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 4 deletions.
diff --git a/Sources/Clients/AccountPortfoliosClient/AccountPortfoliosClient+Live.swift b/Sources/Clients/AccountPortfoliosClient/AccountPortfoliosClient+Live.swift
@@ -803,7 +803,7 @@ extension AccountPortfoliosClient {
 
 extension AccountPortfoliosClient {
 	@Sendable static func extractTags(item: GatewayAPI.StateEntityDetailsResponseItem) -> [AssetTag] {
-		item.metadata.tags?.map(AssetTag.init) ?? []
+		item.metadata.tags?.compactMap(NonEmptyString.init(rawValue:)).map(AssetTag.init) ?? []
 	}
 }
 

diff --git a/Sources/Core/SharedModels/Assets/AssetTag.swift b/Sources/Core/SharedModels/Assets/AssetTag.swift
@@ -1,13 +1,14 @@
 import Foundation
+import NonEmpty
 
 // MARK: - AssetTag
 public enum AssetTag: Hashable, Sendable, Codable {
 	case officialRadix
-	case custom(String)
+	case custom(NonEmptyString)
 }
 
 extension AssetTag {
-	public init(_ customString: String) {
+	public init(_ customString: NonEmptyString) {
 		self = .custom(customString)
 	}
 }
diff --git a/Sources/Features/AssetsFeature/Components/HelperViews/AssetTagsView.swift b/Sources/Features/AssetsFeature/Components/HelperViews/AssetTagsView.swift
@@ -49,7 +49,7 @@ extension AssetTag {
 			return "Official Radix" // FIXME: Strings
 
 		case let .custom(string):
-			return string
+			return string.rawValue
 		}
 	}