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

feat: add support for current sidechain design #397

Closed
wants to merge 22 commits into from

Conversation

mvadari
Copy link
Collaborator

@mvadari mvadari commented Jun 2, 2022

High Level Overview of Change

This PR adds support for the latest version of the sidechain design. It also removes the helper methods that were used for the previous version, since those are no longer needed.

There are 6 new transaction types:

  • XChainCreateBridge (creates a new cross-chain bridge)
  • XChainCreateClaimId (creates a new cross-chain claim ID)
  • XChainCommit (locks/burns the funds on the source chain)
  • XChainClaim (claims the transfer funds on the destination chain)
  • XChainAddAttestation (submits an attestation from a witness server)
  • new account stuff (TBD)

Context of Change

sidechain prod release

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

Before / After

Test Plan

Tests were added for all the new features. CI passes.

Future Tasks

  • Better documentation for the new models
  • Add more tests

@mvadari mvadari changed the title Draft feat: add support for current sidechain design feat: add support for current sidechain design Jun 2, 2022
Base automatically changed from fix-binary-codec-types to master June 13, 2022 21:23
@mvadari mvadari changed the base branch from master to remove-old-sidechains June 16, 2022 22:05
@mvadari mvadari changed the base branch from remove-old-sidechains to xrpl-codec-gen July 11, 2022 17:49
@mvadari
Copy link
Collaborator Author

mvadari commented Jul 11, 2022

Replaced with #417

@mvadari mvadari closed this Jul 11, 2022
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.

1 participant