-
Notifications
You must be signed in to change notification settings - Fork 37
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
Integrate Keystone #1911
Integrate Keystone #1911
Conversation
app/src/test/java/io/gnosis/safe/ui/transactions/details/TransactionDetailsViewModelTest.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/io/gnosis/safe/ui/settings/owner/intro/OwnerInfoKeystoneFragment.kt
Outdated
Show resolved
Hide resolved
Found issues: |
|
Updated to latest, please have a look again. |
@zhiying-fan , any progress for point 2(the issue with the signing QR code) ? |
@liliya-soroka I'm looking at the issue, and the new issues as well. |
@zhiying-fan could you please sign the CLA by commenting |
I have read the CLA Document and I hereby sign the CLA |
97d1202
to
17c4484
Compare
…the AlertDialog is not working
Description
This PR include two parts of work:
Changes
For adding owner key
OwnerInfoKeystoneFragment
to start the QR code scanner.KeystoneOwnerSelectionFragment
to show public keys.PublicAddressDerivator
to derive public keys.Owner
namedsourceFingerprint
and migrated the database to version 8.For signing
KeystoneRequestSignatureFragment
for the UI of signing,KeystoneSignViewModel
to handle the logic.CONFIRMATION
REJECTION
INITIATE_TRANSFER
The logic for signing other signing mode in the future is also implemented, which is included in
KeystoneSignViewModel
.Dependencies
TBD
Tracking events name
@gnosis/mobile-devs