-
Notifications
You must be signed in to change notification settings - Fork 219
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!: new tari address scheme #6353
feat!: new tari address scheme #6353
Conversation
Test Results (Integration tests) 2 files 11 suites 23m 0s ⏱️ For more details on these failures, see this check. Results for commit 2872f71. ♻️ This comment has been updated with latest results. |
Test Results (CI) 3 files 120 suites 42m 29s ⏱️ For more details on these failures, see this check. Results for commit 2872f71. ♻️ This comment has been updated with latest results. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this change also make it so in the case of ledger wallets, ledger will now be required to spend coinbases? Because that's a win if I'm reading it correctly.
fb061b8
to
dc80584
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi there. Looking good. I have comments about applicability of the address types.
base_layer/wallet/src/utxo_scanner_service/uxto_scanner_service_builder.rs
Outdated
Show resolved
Hide resolved
Description --- split message to have dedicated to and from fields Motivation and Context --- This is needed for #6353 When a chat client receives a message, it needs to know the complete address of the receiver, not just a part of it. This allows the user to get the complete address by saving what the sender said their address is --------- Co-authored-by: Hansie Odendaal <39146854+hansieodendaal@users.noreply.github.com>
Co-authored-by: Hansie Odendaal <39146854+hansieodendaal@users.noreply.github.com>
a04fbac
to
ebba5f2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
utACK
Description
Changes the tari address scheme to support view keys and spend keys.
Makes network and features human-readable
Motivation and Context
This allows Tari to support stealth one-sided transactions via Hardware devices.
Makes features and network human readable.
How Has This Been Tested?