backend/handlers: fix header sync tx timestamp

The use of creation timestamp when the tx timestamp was not available
caused the app to show the current date for all the btc/ltc transactions
during the header syncing. This removes the use of the tx creation
timestamp, as it doesn't seem to be really useful anyway.

Now during header sync the tx date is not showed at all.

Author: Beerosagos

CHANGELOG.md

@@ -7,6 +7,7 @@
 - Gracefully shut down Electrum connections upon closing the app
 - Show the selected coin's address on the confirmation screen
 - Fetch balances of multiple ETH accounts at the same time, instead of one by one.
+- Fix wrong btc/ltc transaction timestamp during header sync
 
 ## v4.48.3
 - Linux: fix compatiblity with some versions of Mesa also when using the AppImage

backend/coins/btc/handlers/handlers.go

@@ -140,9 +140,6 @@ func (handlers *Handlers) getTxInfoJSON(txInfo *accounts.TransactionData, detail
 	amount := txInfo.Amount.FormatWithConversions(handlers.account.Coin(), false, accountConfig.RateUpdater)
 	var formattedTime *string
 	timestamp := txInfo.Timestamp
-	if timestamp == nil {
-		timestamp = txInfo.CreatedTimestamp
-	}
 
 	deductedAmountAtTime := txInfo.DeductedAmount.FormatWithConversionsAtTime(handlers.account.Coin(), timestamp, accountConfig.RateUpdater)
 	amountAtTime := txInfo.Amount.FormatWithConversionsAtTime(handlers.account.Coin(), timestamp, accountConfig.RateUpdater)