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

Support CrosschainLog #1144

Conversation

quantumagi
Copy link
Contributor

@quantumagi quantumagi commented May 29, 2023

Adds ability to Interflux to incorporate destination account and network from a CrosschainLog. SC usage is as follows:

    public void MintWithMetadataForNetwork(Address account, UInt256 amount, string metadata, string destinationAccount, string destinationNetwork)
    {
        MintWithMetadata(account, amount, metadata);

        // Perform a cross-chain transfer.
        if (TransferFrom(account, Interflux, amount))
        {
            Log(new CrosschainLog()
            {
                Address = destinationAccount,
                Network = destinationNetwork
            });
        }
    }

@quantumagi quantumagi changed the base branch from release/1.4.0.7 to feature/1.5.0.0-interflux May 29, 2023 05:54
@quantumagi quantumagi requested a review from zeptin May 29, 2023 06:06
@quantumagi quantumagi marked this pull request as draft May 29, 2023 07:44
@quantumagi quantumagi changed the title Support network suffix in burn metadata Support CrosschainLog May 29, 2023
@quantumagi quantumagi requested a review from noescape00 May 29, 2023 08:18
@quantumagi quantumagi marked this pull request as ready for review May 29, 2023 08:27
@zeptin
Copy link
Collaborator

zeptin commented Jun 19, 2023

This looks good, but maybe the changes from #1145 should be incorporated into it as well, as there is some overlap.

@quantumagi quantumagi merged commit c298f63 into stratisproject:feature/1.5.0.0-interflux Jun 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants