Skip to content
This repository has been archived by the owner on Mar 12, 2024. It is now read-only.

Payable vouchers #47

Closed
wants to merge 2 commits into from
Closed

Payable vouchers #47

wants to merge 2 commits into from

Conversation

guidanoli
Copy link
Contributor

@guidanoli guidanoli commented May 22, 2023 •
edited
Loading

@guidanoli guidanoli added feature New feature or request onchain Related to the onchain code labels May 22, 2023
@guidanoli guidanoli self-assigned this May 22, 2023
@gligneul gligneul changed the base branch from next to main May 25, 2023 14:27
@ZzzzHui ZzzzHui mentioned this pull request Sep 26, 2023
Closed
@guidanoli guidanoli force-pushed the feature/payable-vouchers branch 2 times, most recently from 1f7a6bd to afeafd9 Compare July 18, 2023 18:22
@guidanoli guidanoli changed the base branch from main to feature/output-unification July 18, 2023 18:24
@guidanoli
feat(contracts)!: allow vouchers to transfer Ether
82af51b 
* Add an `executeVoucher` function to `ICartesiDApp` and `CartesiDApp`
  with an extra `uint256` parameter called `_value`, which is the amount
  of Wei to be sent along the message call. Backwards compatibility is
  kept with the old signature by passing 0 Wei by default.

* Change the encoding of vouchers to have an extra `value` field.

BREAKING CHANGE: Changes the encoding of vouchers.
@guidanoli
test(contracts): adjust tests to payable vouchers
7782098
@guidanoli
Copy link
Contributor Author

Rebased.

@guidanoli
Copy link
Contributor Author

This PR has been migrated to cartesi/rollups-contracts#45.

@guidanoli guidanoli closed this Aug 28, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees

@guidanoli guidanoli

Labels
feature New feature or request onchain Related to the onchain code
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@guidanoli