Skip to content

Trigger a TUI alert for transactions with an output belonging to this wallet #11016

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.

Sign up for GitHub

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

Closed
TheButterZone wants to merge 2 commits into from
Closed

Trigger a TUI alert for transactions with an output belonging to this wallet #11016

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
5554bff
Add beep on TUI wallet update
TheButterZone Oct 5, 2026
b1ab668
Add alert for transactions with an output belonging to this wallet
TheButterZone Oct 6, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 15 additions & 0 deletions electrum/gui/text.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -119,6 +119,21 @@ def stop(self):
def on_event_wallet_updated(self, wallet):
self.need_update = True

@event_listener
def on_event_new_transaction(self, wallet, tx):

@f321x f321x Oct 6, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This could become annoying as it will beep while the wallet is detecting transactions during sync.
You could duplicate the pattern the other UIs are using, see Qt:

electrum/electrum/gui/qt/main_window.py

Line 922 in cdf9ceb

def notify_transactions(self):

then maybe trigger the notification from the getch() loop?

If you can find a way to deduplicate some of the notification logic between the three UIs this might be even better.

@TheButterZone TheButterZone Oct 6, 2026 •
edited
Loading

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I doubt I can, it took too long with the increasingly-dumb/mislabeled LLMs & eventually having them fight each other just to get to this tiny bit of code.

P.S. I find the multiple beeps during sync satisfying, because when they stop, I know it's time to double-check that they're all done.

if wallet != self.wallet:
return

self.need_update = True

# Alert for transactions with an output belonging to this wallet.
if any(
output.address
and self.wallet.is_mine(output.address)
for output in tx.outputs()
):
curses.beep()

@event_listener
def on_event_network_updated(self):
self.need_update = True
Expand Down