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(ffi)!: add features, metadata_signature and sender_offset_public_key to import_utxo #3767

Merged

Conversation

sdbondi
Copy link
Member

@sdbondi sdbondi commented Jan 28, 2022

Description

  • adds two arguments to wallet_import_utxo (features, metadata_signature and sender_offset_public_key)

Motivation and Context

Allows faucet utxos to be imported to the wallet db correctly

How Has This Been Tested?

@sdbondi sdbondi force-pushed the wallet-import-utxo-sopk branch from d8de2e6 to 595e50e Compare January 28, 2022 08:22
@sdbondi sdbondi changed the title feat(ffi)!: add metadata_signature and sender_offset_public_key to import_utxo feat(ffi)!: add features, metadata_signature and sender_offset_public_key to import_utxo Jan 28, 2022
@sdbondi sdbondi force-pushed the wallet-import-utxo-sopk branch from 595e50e to bea1748 Compare January 28, 2022 08:24
SWvheerden
SWvheerden previously approved these changes Jan 28, 2022
Copy link
Collaborator

@SWvheerden SWvheerden left a comment

Choose a reason for hiding this comment

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

This looks good.
I would add script_key and covenants now as well as require inputs.
We leave todo's here to not break the mobile builds. But as we are requiring more info here we might as well add them now.

@sdbondi
Copy link
Member Author

sdbondi commented Jan 28, 2022

@SWvheerden ok so I think we may as well do that, and if the wallet passes null in we use defaults - wasnt sure if adding new types to ffi is a pain or not - but looks like it's not a problem in unsafe land :)

Copy link
Contributor

@hansieodendaal hansieodendaal left a comment

Choose a reason for hiding this comment

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

Just the one comment please

base_layer/wallet_ffi/src/lib.rs Show resolved Hide resolved
hansieodendaal
hansieodendaal previously approved these changes Jan 28, 2022
Copy link
Contributor

@hansieodendaal hansieodendaal left a comment

Choose a reason for hiding this comment

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

LGTM

@aviator-app aviator-app bot merged commit 7d8aa69 into tari-project:development Jan 28, 2022
@sdbondi sdbondi deleted the wallet-import-utxo-sopk branch January 28, 2022 09:53
sdbondi added a commit to sdbondi/tari that referenced this pull request Jan 31, 2022
* development: (29 commits)
  feat(base-node): add base node prometheus metrics (tari-project#3773)
  fix(ffi): mut pointers should be const (tari-project#3775)
  chore: update launchpad for dibbler (tari-project#3769)
  v0.27.3
  fix(ffi): missing param in header.h (tari-project#3774)
  v0.27.2
  v0.27.2
  fix(ffi): fix bad access (tari-project#3772)
  chore(collectibles): fix eslint
  v0.27.1
  fix(ffi): fix null pointer assignment in import_utxo (tari-project#3770)
  fix(validators): check input metadata matches spending output (tari-project#3761)
  v0.27.0
  feat(ffi)!: Add commitment_signature_create and destroy (tari-project#3768)
  feat(ffi)!: add features, metadata_signature and sender_offset_public_key to import_utxo (tari-project#3767)
  feat(explorer): better view on mempool (tari-project#3763)
  feat: update the available balance in console wallet (tari-project#3760)
  chore: reclassify very long log message (tari-project#3752)
  fix: fix cucumber test for standard recovery (tari-project#3757)
  style: format with prettier (same settings as in integration tests) (tari-project#3756)
  ...
@sdbondi sdbondi restored the wallet-import-utxo-sopk branch February 3, 2022 05:28
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.

4 participants