-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
GUI displays only one xpub for multisig wallet #4777
Comments
Does this wallet actually work? It seems all the xpubs are the same (I don't think it's a display bug). Try opening the wallet file on disk with a text editor, and search for When you create a new wallet, can you reproduce this problem there? |
Yes, it's display issue. Wallet works - I can sign with all my hardware devices.
The same happens if you do software multisig, all xpubs are in the file but they are not displayed by the GUI. You can just try to set up multisig address by yourself with random xpubs and reproduce this issue. Only the first one is shown. |
Yes well, it's not that simple :) Before asking, I've already tried. I see you are on a Mac; I can only test atm on Linux and Windows. I could not reproduce on either. It would be nice if someone else could try to reproduce on MacOS. |
I was running an official binary from electrum.org Just tried on Windows - everything works perfectly. I will try on my other mac laptop at home and write you back. I am running on macOS High Sierra 10.13.6 |
If you can run from source, maybe try: diff --git a/electrum/gui/qt/main_window.py b/electrum/gui/qt/main_window.py
index 5be84beca..8699fe37f 100644
--- a/electrum/gui/qt/main_window.py
+++ b/electrum/gui/qt/main_window.py
@@ -2094,6 +2094,7 @@ class ElectrumWindow(QMainWindow, MessageBoxMixin, PrintError):
mpk_text.addCopyButton(self.app)
def show_mpk(index):
mpk_text.setText(mpk_list[index])
+ mpk_text.update()
# only show the combobox in case multiple accounts are available
if len(mpk_list) > 1:
def label(key): |
Thanks for pointing out where to look =) This helped: def show_mpk(index):
mpk_text.setText(mpk_list[index])
mpk_text.repaint() |
So to make sure, The Qt docs suggest using re
Heh, I guess you could try checking |
|
I updated pyQt to latest version (5.11.3) and everything works now, without any changes in the code. |
Yes this is expected, the binary has a fixed version of PyQt5 bundled; it won't use the one installed on the system. Thanks for investigating. We could just bump the version of PyQt5 we bundle, but unfortunately that would also raise the min supported MacOS version. See #3685 (comment) |
Note: same thing in the wizard during the script type/derivation dialog (selecting radio button does not refresh text). |
Note: the situation is a bit more complicated than just the Qt version.
EDIT: whether you have the "dark" theme (in app) enabled also factors in... argh |
got new report on IRC for this....
display bug... |
Ok I've tested on macOS 10.14.4
So I guess we should update Python in the binary... |
Experiencing the same bug.
The file is encrypted. Is there a way to list the master pub keys using command line or the Electrum console? |
@sunknudsen run this in the
|
Thanks @SomberNight, works like a charm. |
I have reproduced this is 3.3.6 on macos. The xpubs are different in the wallet file but show the same in the UI. |
When using multisig wallet (no matter with software or hardware key storage) I can't see master public keys of the co-signers. Only the very first one is displayed.
The text was updated successfully, but these errors were encountered: