A bunch of bugs #1054
A bunch of bugs #1054
Conversation
|
I have a better solution for the id, ResourceBalance shouldn't be ID:ed at all really, it's only in the TransactionReview context that we use that, and that's really only so that we can apply guarantees. I use About the other bugs, what are they? |
There was a problem hiding this comment.
applied the proper forcerRefresh.
| @@ -4,7 +4,7 @@ public typealias TXID = TransactionHash | |||
|
|
|||
| extension TXID { | |||
| public func formatted(_ format: AddressFormat = .default) -> String { | |||
| bytes().hex() | |||
| asStr() | |||
There was a problem hiding this comment.
Properly formatted TXID
| @@ -77,7 +77,7 @@ public struct AssetTransfer: Sendable, FeatureReducer { | |||
| let manifest = try await createManifest(accounts) | |||
| Task { | |||
| _ = try await dappInteractionClient.addWalletInteraction( | |||
| .transaction(.init(send: .init(transactionManifest: manifest))), | |||
| .transaction(.init(send: .init(transactionManifest: manifest, message: message))), | |||
There was a problem hiding this comment.
Message was removed in some of older PRs
| @@ -90,11 +90,11 @@ private extension View { | |||
| private func addAsset(with destinationStore: PresentationStoreOf<TransferAccountList.Destination>) -> some View { | |||
| sheet(store: destinationStore.scope(state: \.state.addAsset, action: \.addAsset)) { assetsStore in | |||
| AssetsView.View(store: assetsStore) | |||
| .navigationTitle(L10n.AssetTransfer.AddAssets.navigationTitle) | |||
There was a problem hiding this comment.
With latest iOS updates the order matters now.
Several small bug fixes: