Skip to content
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

[#79] Refund transaction documentation #110

Merged
merged 1 commit into from
Mar 27, 2018

Conversation

szerepak
Copy link

No description provided.

@szerepak szerepak force-pushed the cl-79-update-documentation-refund-transaction branch 2 times, most recently from 9e1c0e5 to ab180e0 Compare March 13, 2018 11:25
'secret_hash': 'ce4209765f76e8c29bbcdc26c33746a0dc6feebe986444f8326008808ece2c33',
'size': 255,
'size_text': '255 bytes',
'transaction_hash': 'cc37c5a6f66f24b33e85c5ac2210488873ac61dad4e3098d7a6b35f7f55ff579',
Copy link
Contributor

Choose a reason for hiding this comment

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

transaction_hash was changed 3 days ago (d237ce7) to transaction_address + contract_address was added.

@szerepak szerepak force-pushed the cl-79-update-documentation-refund-transaction branch from ab180e0 to 2bdbe1d Compare March 19, 2018 14:43
@szerepak szerepak changed the title [#79] Refund transaction documentation [WIP] [#79] Refund transaction documentation Mar 19, 2018
@szerepak szerepak force-pushed the cl-79-update-documentation-refund-transaction branch 3 times, most recently from 855a66e to 25f272a Compare March 21, 2018 14:32
@szerepak szerepak changed the title [WIP] [#79] Refund transaction documentation [#79] Refund transaction documentation Mar 21, 2018
@szerepak szerepak requested a review from pafff March 21, 2018 14:32
@szerepak
Copy link
Author

@trojkat @pafff This is ready to review :) Both refund transaction went well!

@szerepak szerepak force-pushed the cl-79-update-documentation-refund-transaction branch from 25f272a to 4b18be6 Compare March 21, 2018 14:33
@@ -86,7 +86,7 @@ def redeem(self, wallet, secret):
def refund(self, wallet):
if self.locktime > datetime.utcnow():
locktime_string = self.locktime.strftime('%Y-%m-%d %H:%M:%S')
raise RuntimeError(f"This contract is still valid! It can't be refunded until {locktime_string}.")
raise RuntimeError(f"This contract is still valid! It can't be refunded until {locktime_string}. UTC")
Copy link
Contributor

Choose a reason for hiding this comment

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

Move the dot to the end.

refund_transaction = alice_contract.refund(alice_ltc_wallet)
RuntimeError: This contract is still valid! It can't be refunded until 2018-03-21 14:27:19 UTC.

Oops! Something wen wrong! Contract is still valid! Alice was a bit impatient, wasn't she?
Copy link
Contributor

Choose a reason for hiding this comment

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

went

refund_transaction = bob_contract.refund(bob_mona_wallet)
RuntimeError: This contract is still valid! It can't be refunded until 2018-03-20 14:37:01 UTC.

Oops! Something wen wrong! Contract is still valid! Bob was a bit impatient, wasn't he?
Copy link
Contributor

Choose a reason for hiding this comment

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

went

@szerepak szerepak force-pushed the cl-79-update-documentation-refund-transaction branch from 4b18be6 to f09f4d6 Compare March 21, 2018 14:48
@trojkat trojkat force-pushed the cl-79-update-documentation-refund-transaction branch from f09f4d6 to 26cd513 Compare March 27, 2018 10:13
@trojkat trojkat merged commit 06d949a into master Mar 27, 2018
@trojkat trojkat deleted the cl-79-update-documentation-refund-transaction branch March 27, 2018 10:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants