Skip to content

Commit

Permalink
Don't crash on contracts with no sender specific message
Browse files Browse the repository at this point in the history
Commit 470fbba didn't take this case
into account, assuming it could only be empty and not None, but it's an
optional parameter tot he function and the "Send test contract"
functionality explicitly does that.

Spotted by Steve Singer
  • Loading branch information
mhagander committed Nov 5, 2024
1 parent 39cbdeb commit 2453bc2
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion postgresqleu/digisign/implementations/signwell.py
Original file line number Diff line number Diff line change
Expand Up @@ -113,7 +113,7 @@ def send_contract(self, sender_name, sender_email, recipient_name, recipient_ema
"id": "2",
"name": sender_name,
"email": sender_email,
"message": message_to_sender.replace("\n", "<br/>") or message.replace("\n", "<br/>"),
"message": message_to_sender.replace("\n", "<br/>") if message_to_sender else message.replace("\n", "<br/>"),
},
],
"apply_signing_order": True,
Expand Down

0 comments on commit 2453bc2

Please sign in to comment.