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

Implement cross chain command for interoperability modules - Closes #7038 & #7039 #7165

Conversation

ishantiw
Copy link
Contributor

What was the problem?

This PR resolves #7038 & #7039

How was it solved?

♻️ Export calculateRootFromRightWitness from lisk-tree

🌱 Implement ccu on mainchain

🌱 Implement CCU command on sidechain

How was it tested?

npm run test:unit interoperability

@ishantiw ishantiw force-pushed the 7038-7039-impl-ccu-mainchain branch from 588f89e to 4f191c2 Compare May 16, 2022 07:41
@ishantiw ishantiw requested a review from shuse2 May 16, 2022 07:41
@ishantiw ishantiw force-pushed the 7038-7039-impl-ccu-mainchain branch from 4f191c2 to 12d04d6 Compare May 16, 2022 08:02
Copy link
Collaborator

@shuse2 shuse2 left a comment

Choose a reason for hiding this comment

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

Looking good in general!
I think we should ensure empty check for messageWitness, certificate and inboxUpdate which is bit confusing.

framework/src/modules/interoperability/utils.ts Outdated Show resolved Hide resolved
@ishantiw ishantiw marked this pull request as ready for review May 19, 2022 16:16
@ishantiw ishantiw force-pushed the 7038-7039-impl-ccu-mainchain branch from 209aaf8 to 11d57fe Compare May 23, 2022 11:51
@shuse2 shuse2 requested review from shuse2 and Incede May 23, 2022 11:53
@ishantiw ishantiw requested a review from shuse2 May 24, 2022 13:16
framework/src/modules/interoperability/utils.ts Outdated Show resolved Hide resolved
framework/src/modules/interoperability/utils.ts Outdated Show resolved Hide resolved
framework/src/modules/interoperability/utils.ts Outdated Show resolved Hide resolved
framework/src/modules/interoperability/utils.ts Outdated Show resolved Hide resolved
@ishantiw ishantiw requested a review from shuse2 May 25, 2022 08:03
Copy link
Collaborator

@shuse2 shuse2 left a comment

Choose a reason for hiding this comment

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

well done 🚀

@ishantiw ishantiw force-pushed the 7038-7039-impl-ccu-mainchain branch from d128dea to 2cfb4b6 Compare May 27, 2022 09:09
@ishantiw ishantiw merged commit 3915d86 into feature/6915-introduce-interoperability-module May 27, 2022
@ishantiw ishantiw deleted the 7038-7039-impl-ccu-mainchain branch May 27, 2022 13:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
3 participants