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

CU-1yrzbe3 - Bonded finance beneficiary #443

Merged
merged 7 commits into from
Jan 5, 2022

Conversation

hussein-aitlahcen
Copy link
Contributor

  • Introduce a new beneficiary field for an offer, effectively allowing the pallet to transfer funds to the beneficiary when the maturity of an offer is Infinite
  • Fix a case where the cancellation would result in the remaining reward not being refunded to the issuer
  • Added test cases to cover the new changes

@itsbobbyzz
Copy link

Task linked: CU-1yrzbe3 Forever Bonds

dzmitry-lahoda
dzmitry-lahoda previously approved these changes Jan 4, 2022
Copy link
Contributor

@dzmitry-lahoda dzmitry-lahoda left a comment

Choose a reason for hiding this comment

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

runtime code looks ok

benluelo
benluelo previously approved these changes Jan 4, 2022
@KaiserKarel KaiserKarel merged commit a150c48 into main Jan 5, 2022
@KaiserKarel KaiserKarel deleted the bonded-finance-beneficiary branch January 5, 2022 11:38
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.

5 participants