-
Notifications
You must be signed in to change notification settings - Fork 492
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Step forward to Remove TransactionSummary
(1/n)
#11520
Step forward to Remove TransactionSummary
(1/n)
#11520
Conversation
@@ -17,7 +17,7 @@ public partial class CoinJoinHistoryItemViewModel : HistoryItemViewModelBase | |||
bool isSingleCoinJoinTransaction) | |||
: base(orderIndex, transactionSummary) | |||
{ | |||
Date = transactionSummary.DateTime.ToLocalTime(); | |||
Date = transactionSummary.FirstSeen.ToLocalTime(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
SmartTransaction
has FirstSeen
property, so rename it.
@@ -78,4 +79,7 @@ private static IEnumerable<BitcoinAddress> GetDestinationAddresses(ICollection<I | |||
// All outputs that are not my own are the destinations. | |||
return foreignOutputs.Select(x => x.DestinationAddress); | |||
} | |||
|
|||
public static int GetConfirmations(this SmartTransaction transaction, int blockchainTipHeight) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Extracted from TransactionSummaryExtensions.cs
@@ -123,7 +122,7 @@ public static bool NotNullAndNotEmpty<T>(this IEnumerable<T> source) | |||
} | |||
} | |||
} | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some whitespace from a previous PR.
… original implementation
1ee5a02
to
e660115
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tACK, changes look trivial and correct. IMO mergeable once the CI is finished.
TransactionSummary
(1/n)TransactionSummary
(1/n)
This is another PR to simplify #11458.
Easy to review commit by commit.