Skip to content

Commit

Permalink
Merge pull request #578 from BamaHodl/MethodAndPropertyIssue
Browse files Browse the repository at this point in the history
Fix issue caused by having is_sign_message method and property of same name
  • Loading branch information
newtonick authored Jul 19, 2024
2 parents 13e1d60 + bef2267 commit 5e4f669
Showing 1 changed file with 1 addition and 6 deletions.
7 changes: 1 addition & 6 deletions src/seedsigner/models/decode_qr.py
Original file line number Diff line number Diff line change
Expand Up @@ -382,7 +382,7 @@ def detect_segment_type(s, wordlist_language_code=None):
return QRType.BITCOIN_ADDRESS

# message signing
elif DecodeQR.is_sign_message(s):
elif s.startswith("signmessage"):
return QRType.SIGN_MESSAGE

# config data
Expand Down Expand Up @@ -506,11 +506,6 @@ def is_bitcoin_address(s):
return False


@staticmethod
def is_sign_message(s):
return type(s) == str and s.startswith("signmessage")


@staticmethod
def multisig_setup_file_to_descriptor(text) -> str:
# sample text file, parse the contents and create descriptor
Expand Down

0 comments on commit 5e4f669

Please sign in to comment.