diff --git a/electrum/gui/qml/qetransactionlistmodel.py b/electrum/gui/qml/qetransactionlistmodel.py
index b93ff505c..dc4175c1b 100644
--- a/electrum/gui/qml/qetransactionlistmodel.py
+++ b/electrum/gui/qml/qetransactionlistmodel.py
@@ -127,7 +127,7 @@ class QETransactionListModel(QAbstractListModel, QtEventListener):
         #self._logger.debug(str(tx_item))
         item = tx_item
 
-        item['key'] = item.get('txid') or item.get('group_id') or item['payment_hash']
+        item['key'] = item.get('txid') or item['payment_hash'] or item['group_id'] # fixme: this is fragile
 
         if 'lightning' not in item:
             item['lightning'] = False
diff --git a/electrum/lnworker.py b/electrum/lnworker.py
index 693d07734..f231e679d 100644
--- a/electrum/lnworker.py
+++ b/electrum/lnworker.py
@@ -1009,7 +1009,7 @@ class LNWallet(LNWorker):
                 timestamp = timestamp or 0,
                 label=label,
             )
-            out[payment_hash] = item
+            out[payment_hash.hex()] = item
         for chan in itertools.chain(self.channels.values(), self.channel_backups.values()):  # type: AbstractChannel
             item = chan.get_funding_height()
             if item is None: