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

[wallet] Add send one-sided tx to console wallet TUI (tari-script) #2863

Merged

Conversation

hansieodendaal
Copy link
Contributor

@hansieodendaal hansieodendaal commented Apr 20, 2021

Description

Added the one-sided payment transaction sending functionality to the console wallet TUI.

NOTE: Currently only the sender wallet will be able to monitor the transaction; the receiver wallet scanning the blockchain for one-sided transactions must still be implemented.

Motivation and Context

Expanding the tari script functionality.

How Has This Been Tested?

Tested sending one-sided transactions between console wallets, verified that the sender wallet detected the transactions as mined.

Checklist:

  • I'm merging against the tari-script branch.
  • I have squashed my commits into a single commit.

@hansieodendaal hansieodendaal requested a review from delta1 April 20, 2021 08:13
@hansieodendaal hansieodendaal changed the title Add send one-sided tx to console wallet TUI [wallet] Add send one-sided tx to console wallet TUI (tari-script) Apr 20, 2021
@hansieodendaal hansieodendaal force-pushed the ho_one_sided_tx_tui branch 2 times, most recently from 956e166 to 3c587ba Compare April 20, 2021 14:15
Added the one-sided payment transaction sending functionality to
the console wallet TUI.
TODO: Receiver wallet scanning the blockchain for one-sided
      transactions.
@stringhandler stringhandler merged commit 2e80f29 into tari-project:tari-script Apr 26, 2021
@hansieodendaal hansieodendaal deleted the ho_one_sided_tx_tui branch April 26, 2021 07:02
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.

3 participants