-
Notifications
You must be signed in to change notification settings - Fork 868
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(wallet): Shield ZCash Account Modal #26214
Conversation
eaeae83
to
01c6c75
Compare
A Storybook has been deployed to preview UI for the latest push |
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.
++
@@ -981,6 +981,7 @@ struct WalletInfo { | |||
bool is_z_cash_enabled; | |||
bool is_ankr_balances_feature_enabled; | |||
bool is_transaction_simulations_feature_enabled; | |||
bool is_z_cash_shielded_transactions_enabled; |
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.
nit: zcash
const canShieldAccount = | ||
isZCashShieldedTransactionsEnabled && | ||
account.accountId.coin === BraveWallet.CoinType.ZEC && | ||
zCashAccountInfo && |
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.
I think it is better to use zcash instead of zCash like in other places
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.
++
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.
iOS++
01c6c75
to
a78f2ad
Compare
a78f2ad
to
61a3657
Compare
[puLL-Merge] - brave/brave-core@26214 DescriptionThis PR adds support for ZCash shielded transactions in the Brave Wallet. It introduces new UI elements and functionality to allow users to upgrade their ZCash accounts to shielded accounts, which provide enhanced privacy for transactions. ChangesChanges
Possible Issues
Security HotspotsNo significant security hotspots were identified in this change. However, as this feature deals with privacy-enhancing functionality for cryptocurrency transactions, it's crucial that the implementation correctly handles the shielding process and doesn't inadvertently expose sensitive information. |
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.
strings look good
Released in v1.73.62 |
Description
Introduces the ability to
Shield
a ZCash Account on theAccounts
page.Resolves brave/brave-browser#41780
Submitter Checklist:
QA/Yes
orQA/No
;release-notes/include
orrelease-notes/exclude
;OS/...
) to the associated issuenpm run test -- brave_browser_tests
,npm run test -- brave_unit_tests
wikinpm run presubmit
wiki,npm run gn_check
,npm run tslint
git rebase master
(if needed)Reviewer Checklist:
gn
After-merge Checklist:
changes has landed on
Test Plan:
Wallet
and navigate to theAccounts
tabMore Menu
for you ZCash AccountSwitch to shielded account
Shield account
Shielded
label next to yourAccount Name
.Screen.Recording.5.mov