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

Cw 503 ledger support #1310

Merged
merged 72 commits into from May 1, 2024
Merged

Cw 503 ledger support #1310

merged 72 commits into from May 1, 2024

Conversation

konstantinullrich
Copy link
Contributor

Description

Add support for Ledger Hardware Wallets

Pull Request - Checklist

  • Initial Manual Tests Passed
  • Double check modified code and verify it with the feature/task requirements
  • Format code
  • Look for code duplication
  • Clear naming for variables and methods

# Conflicts:
#	android/app/src/main/AndroidManifestBase.xml
#	cw_bitcoin/lib/bitcoin_wallet_service.dart
#	cw_core/lib/wallet_service.dart
#	cw_monero/lib/monero_wallet_service.dart
#	res/values/strings_ar.arb
#	res/values/strings_bg.arb
#	res/values/strings_cs.arb
#	res/values/strings_de.arb
#	res/values/strings_en.arb
#	res/values/strings_es.arb
#	res/values/strings_fr.arb
#	res/values/strings_ha.arb
#	res/values/strings_hi.arb
#	res/values/strings_hr.arb
#	res/values/strings_id.arb
#	res/values/strings_it.arb
#	res/values/strings_ja.arb
#	res/values/strings_ko.arb
#	res/values/strings_my.arb
#	res/values/strings_nl.arb
#	res/values/strings_pl.arb
#	res/values/strings_pt.arb
#	res/values/strings_ru.arb
#	res/values/strings_th.arb
#	res/values/strings_tl.arb
#	res/values/strings_tr.arb
#	res/values/strings_uk.arb
#	res/values/strings_ur.arb
#	res/values/strings_yo.arb
#	res/values/strings_zh.arb
# Conflicts:
#	cw_bitcoin/lib/bitcoin_wallet_service.dart
#	cw_bitcoin_cash/lib/src/bitcoin_cash_wallet_service.dart
#	cw_core/lib/hive_type_ids.dart
#	cw_core/lib/wallet_base.dart
#	cw_core/lib/wallet_info.dart
#	cw_core/lib/wallet_service.dart
#	cw_ethereum/lib/ethereum_wallet_service.dart
#	cw_monero/lib/monero_wallet_service.dart
#	cw_nano/lib/nano_wallet_service.dart
#	cw_polygon/lib/polygon_wallet_service.dart
#	lib/di.dart
#	lib/ethereum/cw_ethereum.dart
#	lib/src/screens/send/send_page.dart
#	pubspec_base.yaml
#	res/values/strings_de.arb
#	tool/configure.dart
# Conflicts:
#	lib/src/screens/send/send_page.dart
…03-Ledger-support

# Conflicts:
#	cw_evm/lib/evm_chain_client.dart
#	lib/di.dart
#	lib/view_model/send/send_view_model.dart
@konstantinullrich konstantinullrich marked this pull request as ready for review April 9, 2024 11:28
Copy link
Contributor

@OmarHatem28 OmarHatem28 left a comment

Choose a reason for hiding this comment

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

only the UI pages files are left, so will submit my comments so far

cw_evm/pubspec.yaml Outdated Show resolved Hide resolved
cw_evm/pubspec.yaml Show resolved Hide resolved
cw_evm/lib/evm_ledger_credentials.dart Outdated Show resolved Hide resolved
lib/buy/dfx/dfx_buy_provider.dart Show resolved Hide resolved
lib/view_model/send/send_view_model.dart Outdated Show resolved Hide resolved
cw_evm/lib/evm_ledger_credentials.dart Show resolved Hide resolved
lib/buy/dfx/dfx_buy_provider.dart Outdated Show resolved Hide resolved
lib/ethereum/cw_ethereum.dart Show resolved Hide resolved
lib/ethereum/cw_ethereum.dart Show resolved Hide resolved
pubspec_base.yaml Outdated Show resolved Hide resolved
konstantinullrich and others added 3 commits April 30, 2024 12:29
# Conflicts:
#	cw_core/lib/hive_type_ids.dart
#	cw_core/lib/wallet_credentials.dart
#	cw_core/lib/wallet_info.dart
#	lib/main.dart
#	lib/view_model/wallet_creation_vm.dart
@OmarHatem28 OmarHatem28 changed the base branch from main to hardware_wallets May 1, 2024 23:52
@OmarHatem28 OmarHatem28 merged commit 4fc7b2a into hardware_wallets May 1, 2024
1 check passed
@OmarHatem28 OmarHatem28 deleted the CW-503-Ledger-support branch May 1, 2024 23:52
@OmarHatem28 OmarHatem28 restored the CW-503-Ledger-support branch May 2, 2024 00:10
@konstantinullrich konstantinullrich deleted the CW-503-Ledger-support branch May 2, 2024 06:03
@konstantinullrich konstantinullrich restored the CW-503-Ledger-support branch May 2, 2024 06:10
OmarHatem28 added a commit that referenced this pull request May 5, 2024
* CW-503 Prepare Ledger integration

* CW-503 Revert Tor ignore

* CW-503 Add Connect Device Page

* CW-503 Add createWalletFromDevice for monero

* CW-503 Add Connect Device Page

* CW-503 Add Connect Device Page

* CW-503 Add Debug Options

* CW-503 Add proper hardware wallet selection screen

* CW-503 Minor design changes on connect_device_page

* CW-503 Add Create Wallet from Ledger

* CW-503 Spent Ledger ETH Funds

* CW-503 Minor fixes

* CW-503 Fix Merge conflicts

* CW-503 Fix Merge conflicts

* CW-503 Fix Merge conflicts

* CW-503 Fix minor conflicts

* CW-503 Improve Ledger BLE Communication

* CW-503 Improve Ledger BLE Communication

* CW-503 Rollback Monero Test code

* CW-503 Fix Execution failed for task :app:checkReleaseDuplicateClasses

* CW-503 Better Error-Exceptions

* CW-503 Add SetPinScreen before restore from hardware-wallet

* CW-503 override web3dart to use cake's git hosted version

* CW-503 Implement ledger sign messages

* CW-503 Implement ledger sign messages and send erc20 tokens

* CW-503 Fix merge conflicts

* CW-503 Fix merge conflicts

* CW-503 Use dep override for ledger_flutter

* CW-503 Ledger ERC20 finalisation

* CW-503 More graceful error handling

* CW-503 Even more graceful error handling & remove debug code

* CW-503 Minor Changes for Vik

* CW-503 Fix USB connection

* CW-503 Maybe this overrides web3dart finally

* Indicate Loading on the first 5 Wallet Accounts

* Prepare Ledger Bitcoin

* Fix conflicts with main

* Add Bluetooth permission to iOS [skip ci]

* add privacyinfo for ios [skip ci]

* update PrivacyInfo.xcprivacy [scip ci]

* ios shit [skip ci]

* Improve bitcoin xpub

* Resolve open Todos regarding iOS Support

* Minor debug

* Remove erc20 dependency to have more granular control over the tx UX

* Create Bitcoin Wallets using xpub

* Create Bitcoin Wallets using xpub

* Better error handling

* Improve Ledger account handling

* Add Bitcoin Support for Ledger

* Add Bitcoin Support for Ledger

* Add Bitcoin Support for Ledger

* Implement requested Changes

* Implement requested Changes

* Minor fix

* Implement requested Changes

* Implement requested Changes

* Add comment to remind me

* Enable RBF for Bitcoin Ledger

* Fix merge conflicts

* Update wallet_info.dart

* Cw 503 ledger support (#1310)

* CW-503 Prepare Ledger integration

* CW-503 Revert Tor ignore

* CW-503 Add Connect Device Page

* CW-503 Add createWalletFromDevice for monero

* CW-503 Add Connect Device Page

* CW-503 Add Connect Device Page

* CW-503 Add Debug Options

* CW-503 Add proper hardware wallet selection screen

* CW-503 Minor design changes on connect_device_page

* CW-503 Add Create Wallet from Ledger

* CW-503 Spent Ledger ETH Funds

* CW-503 Minor fixes

* CW-503 Fix Merge conflicts

* CW-503 Fix Merge conflicts

* CW-503 Fix Merge conflicts

* CW-503 Fix minor conflicts

* CW-503 Improve Ledger BLE Communication

* CW-503 Improve Ledger BLE Communication

* CW-503 Rollback Monero Test code

* CW-503 Fix Execution failed for task :app:checkReleaseDuplicateClasses

* CW-503 Better Error-Exceptions

* CW-503 Add SetPinScreen before restore from hardware-wallet

* CW-503 override web3dart to use cake's git hosted version

* CW-503 Implement ledger sign messages

* CW-503 Implement ledger sign messages and send erc20 tokens

* CW-503 Fix merge conflicts

* CW-503 Fix merge conflicts

* CW-503 Use dep override for ledger_flutter

* CW-503 Ledger ERC20 finalisation

* CW-503 More graceful error handling

* CW-503 Even more graceful error handling & remove debug code

* CW-503 Minor Changes for Vik

* CW-503 Fix USB connection

* CW-503 Maybe this overrides web3dart finally

* Indicate Loading on the first 5 Wallet Accounts

* Fix conflicts with main

* Add Bluetooth permission to iOS [skip ci]

* add privacyinfo for ios [skip ci]

* update PrivacyInfo.xcprivacy [scip ci]

* ios shit [skip ci]

* Resolve open Todos regarding iOS Support

* Remove erc20 dependency to have more granular control over the tx UX

* Better error handling

* Improve Ledger account handling

* Implement requested Changes

* Implement requested Changes

* Implement requested Changes

* Implement requested Changes

* Fix merge conflicts

* Update wallet_info.dart

---------

Co-authored-by: Omar Hatem <omarh.ismail1@gmail.com>

* Fix merge conflicts

* Fix merge conflicts

* Minor Fix to derivations

* Update cw_bitcoin/lib/bitcoin_wallet.dart [skip ci]

* Update cw_bitcoin/lib/bitcoin_wallet.dart [skip ci]

* Update cw_bitcoin/lib/electrum_wallet.dart [skip ci]

* Fix backward compatibility issues

* Merge Tron

* Fix Tron with HW changes

* fix ble on iOS
fix tron address validation

---------

Co-authored-by: Omar Hatem <omarh.ismail1@gmail.com>
@konstantinullrich konstantinullrich deleted the CW-503-Ledger-support branch May 6, 2024 09:48
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.

None yet

2 participants