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

Feature: Multiasset Support #18

Merged
merged 6 commits into from
Jul 18, 2024
Merged

Feature: Multiasset Support #18

merged 6 commits into from
Jul 18, 2024

Conversation

iljabvh
Copy link
Collaborator

@iljabvh iljabvh commented Jul 18, 2024

We extend Stellar payment channels to send and receive multiple SAC token assets.

iljabvh added 6 commits June 19, 2024 16:00
* feat(channel): Adapt channel functionality to support multiple assets on one channel, tested for two assets.

* feat(client): Adapt contract invocation to new version, temporarily hard coding a transaction fee to 500000 stroops.

* feat(payment): Adapt payment channel test/demo to two-asset channels

* feat(testdata): Add multichannel contract, perun_soroban_multi_contract.wasm.

* feat(wire): Adapt encoding between contract types and backend types to multi-asset channels.
…ich the channel is deleted after the second withdrawal (Bob's withdrawal). Hence, GetChannelInfo returns no channel state anymore after a successful withdrawal.

* docs(channel, client): Add logs (temporarily).

* test(payment/test, payment_test): Modify TestHappyPerun to test swap.

* fix(testdata): Update contract perun_soroban_multi_contract.wasm, including the multiasset fixes.
…ance

* refactor(payment_test/client_test.go): Change initial balance to have more general swap case.

* chore(payment/test/client.go): Remove Printing of initAlloc.

* fix(wire/balances.go): Fix balance mismatch, introduced by multiasset feature.
…ding a signed transaction - decode the result of the simulated transaction result
@sophia1ch sophia1ch self-requested a review July 18, 2024 11:42
@iljabvh iljabvh merged commit 71880f4 into dev Jul 18, 2024
1 check failed
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.

2 participants