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.

Sign up for GitHub

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

Jump to bottom

External Wallet #9

Open
wants to merge 9 commits into
base: dev
Choose a base branch
from
Open

External Wallet #9

wants to merge 9 commits into from

Conversation

janbormet
Copy link
Collaborator

This adds support for external wallet that can sign messages by e.g. by RPC.

@janbormet janbormet marked this pull request as draft September 25, 2023 13:27
ndzik and others added 2 commits November 29, 2023 14:54
@ndzik
channel/backend: Use TxView after signing
535da83
@janbormet
channel: Return the native asset in backend.NewAsset as a quick fix
f86091b 
We should follow up to unite the CKBAsset and SUDTAsset types to
allow for generic marshalling of assets as is frequently done
by go-perun.
@janbormet
Copy link
Collaborator Author

janbormet commented Jan 9, 2024 •
edited
Loading

TODO:
Refactor CKBAsset and SUDTAsset into single asset type
See #10

@janbormet
Copy link
Collaborator Author

Closes: #10

@janbormet janbormet marked this pull request as ready for review January 12, 2024 16:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@janbormet @ndzik