-
Notifications
You must be signed in to change notification settings - Fork 18
Multi ledger #337
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
Multi ledger #337
Conversation
16044ca
to
ab2f162
Compare
As I addressed above, I think the general architecture and protocols need to be adapted a little more to actually support multi-ledger channels. Architecture-wise, it looks good to me so far. As I also suggested in a meeting, it would maybe be expedient if we migrated the core to multi-ledger but added a legacy single-ledger adapter for older backends. |
b81018b
to
68375c9
Compare
I have addressed all comments. Significant changes:
|
28c85b5
to
5c0ec67
Compare
Signed-off-by: Matthias Geihs <matthias@perun.network>
Signed-off-by: Matthias Geihs <matthias@perun.network>
Signed-off-by: Matthias Geihs <matthias@perun.network>
Signed-off-by: Matthias Geihs <matthias@perun.network>
1c4df54
to
3190786
Compare
0fad769
to
aeae771
Compare
Addressed all comments. Here is the diff: https://github.com/hyperledger-labs/go-perun/compare/5c0ec670f3b19ab88de2a54925f8fd6575374c42..aeae7717a81d16cd268add2d769f6176814e49c6 Please check and approve if you are happy. @cryptphil @RmbRT |
Signed-off-by: Matthias Geihs <matthias@perun.network>
Signed-off-by: Matthias Geihs <matthias@perun.network>
Signed-off-by: Matthias Geihs <matthias@perun.network>
Signed-off-by: Matthias Geihs <matthias@perun.network>
Signed-off-by: Matthias Geihs <matthias@perun.network>
aeae771
to
c051a07
Compare
Enables multi-ledger channels.
The following changes are applied:
LedgerID
.