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.

Sign up for GitHub

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

Jump to bottom

Zano #1793

Open
wants to merge 124 commits into
base: main
Choose a base branch
from
Open

Zano #1793

wants to merge 124 commits into from

Conversation

MrCyjaneK
Copy link
Collaborator

Description

Early WIP, do not review. yet.

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

@LeoBlackCat
my experiments
70b1241
@LeoBlackCat
Inital code for Zano integration
f6485d9
@LeoBlackCat
Added missing android log lib
7a00a5d
@LeoBlackCat
added dummy wallet & some zano implementation
219f43a
@LeoBlackCat
fixing api for zano
a260d0a
@LeoBlackCat
fixed zano build script
d5f84fe
@LeoBlackCat
attempt tp fix namespace problem
6d7f2be
@LeoBlackCat
added copy script for Zano files
9265651
@LeoBlackCat
changes for zano wallet
58e5059
@LeoBlackCat
last updates
dda6d4c
@LeoBlackCat
zano.dart test app
2d886e1
@LeoBlackCat
wallet recovery
8d3dd6f
@LeoBlackCat
added pending transfer, some cleanup of unused
3eaec03
@LeoBlackCat
some cleanup
df984ca
@LeoBlackCat
send + receive qr code
e4c775b
@LeoBlackCat
last upd
1418679
@LeoBlackCat
updated build_zano.sh
20cf071
@LeoBlackCat
updated zano ApiCalls, removed dummy
00a5bd9
@LeoBlackCat
updated zano ApiCalls, removed dummy
3c2953a
@LeoBlackCat
added logging for get status/get info
e1da71a
@LeoBlackCat
restored old wallet.dart
7fcdc8c
@LeoBlackCat
restoring original versions of files
b3e453a
@LeoBlackCat
restoring original versions of files
d684be6
@LeoBlackCat
restored get_height_by_date.dart, removed unnecessary calls for zano …
b585da5 
…get height
@LeoBlackCat
restoring original versions
ba2dde7
@LeoBlackCat
added multiple destinations, send all flag; some refactoring
43e9cd2
@LeoBlackCat
logging
9228d91
@LeoBlackCat
removed the duplicate
75f1f3f
@LeoBlackCat
fixed syncing sync status, decimal division, safe null json parsing
23485a4
@LeoBlackCat
merge with latest cakewallet to leofork
26e3a6c
@MrCyjaneK MrCyjaneK mentioned this pull request Jan 4, 2025
Closed
5 tasks
@MrCyjaneK MrCyjaneK mentioned this pull request Jan 7, 2025
Open
5 tasks
OmarHatem28
OmarHatem28 requested changes Jan 16, 2025
View reviewed changes
cw_core/lib/crypto_currency.dart Outdated Show resolved Hide resolved
lib/main.dart Outdated Show resolved Hide resolved
lib/entities/provider_types.dart Outdated Show resolved Hide resolved
lib/core/address_validator.dart Outdated Show resolved Hide resolved
ios/.gitignore Outdated Show resolved Hide resolved
scripts/android/build_zano.sh Outdated Show resolved Hide resolved
lib/view_model/dashboard/home_settings_view_model.dart Outdated Show resolved Hide resolved
cw_zano/lib/zano_wallet.dart Show resolved Hide resolved
cw_zano/lib/zano_wallet.dart Outdated Show resolved Hide resolved
cw_zano/lib/zano_wallet.dart Show resolved Hide resolved
@MrCyjaneK
Dynamically detect number of cores used to build monero_c, since it a…
5cbf3ea 
…ppears that zano requires more memory to link (and it reliably fails for first couple builds due to OOM on CI/VM with memory constrains).

Drop unshallowing of all modules
[run tests]
@MrCyjaneK
Merge branch 'main' into zano-pr
d64971c
@MrCyjaneK
Changes from review [run tests]
3d7e4a2
@MrCyjaneK
drop zano on linux (missing symbols)
67c3cd9 
fix wownero on linux
add aarch64-linux-gnu
[run tests]
OmarHatem28
OmarHatem28 reviewed Jan 17, 2025
View reviewed changes
cw_core/lib/node.dart Outdated Show resolved Hide resolved
lib/entities/default_settings_migration.dart Show resolved Hide resolved
lib/view_model/dashboard/home_settings_view_model.dart Outdated Show resolved Hide resolved
lib/view_model/wallet_new_vm.dart Outdated Show resolved Hide resolved
lib/view_model/wallet_restore_view_model.dart Outdated Show resolved Hide resolved
lib/view_model/exchange/exchange_view_model.dart Outdated Show resolved Hide resolved
cw_zano/lib/model/zano_asset.dart Outdated Show resolved Hide resolved
scripts/android/build_zano.sh Outdated Show resolved Hide resolved
windows/CMakeLists.txt Show resolved Hide resolved
@MrCyjaneK
Merge branch 'main' into zano-pr
a5bd8fe
@MrCyjaneK
- remove duplicate entry in addToken()
8ed799e 
- use walletPassword in createZanoNewWalletCredentials
- remove createZanoRestoreWalletFromKeysCredentials
[run tests]
@MrCyjaneK MrCyjaneK requested a review from OmarHatem28 January 19, 2025 09:02
@MrCyjaneK MrCyjaneK mentioned this pull request Jan 19, 2025
Open
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@OmarHatem28 OmarHatem28 Awaiting requested review from OmarHatem28

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@MrCyjaneK @OmarHatem28 @LeoBlackCat @cryptozoidberg