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

Add multi-recipient transactions support #92

Closed
wants to merge 3 commits into from

Conversation

danieloravec
Copy link
Contributor

There are likely some issues with babel fees, even though I tried to address them. No tests were performed.

This draft needs to be extended by some form of communication between individual recipient forms (likely just a parent component passing correctly synchronized props). This is needed so that if there are some assets selected for a recipient A, then these assets are not displayed as fully available for recipient B (available balances are decreased appropriately).

  • Extract recipient form containing
    address and assets into a separate
    component.

  • Keep track of a list of recipients
    in SendView.

  • Extend P2P transaction building such
    that it takes a list of receivers
    instead of just a single one.

  • Successfully sent 2-recipient transaction.

* Extract recipient form containing
  address and assets into a separate
  component.

* Keep track of a list of recipients
  in SendView.

* Extend P2P transaction building such
  that it takes a list of receivers
  instead of just a single one.

* Successfully sent 2-recipient transaction.
@arobsn
Copy link
Member

arobsn commented Jun 4, 2024

This PR is no more compatible with the base branch, closing it for now

@arobsn arobsn closed this Jun 4, 2024
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.

None yet

2 participants