-
Notifications
You must be signed in to change notification settings - Fork 212
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
Zcoin activation date #2356
Zcoin activation date #2356
Conversation
@smk762 I do not observe such entries in logs but code looks working correctly. API (MM2) version in app 1.0.7-beta_79f6205.
All notifications during ARRR activation were displayed correctly. Logs in mm2.log for ARRR:
App log contains following entries for ARRR:
|
Cheers - might be the debug logs not showing in "release" builds. |
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.
ARRR coin can be activated with activation date selection.
ARRR coin could be activated successful. Send and swap for ARRR work as expected. |
Subsequent activations will appear to start from scratch (5%) though will complete faster (if same date setting remains).
so the percentage value is not a pure indication of |
Here's an indication in CLI of sync duration for a few different timestamps within the last 2 weeks.
Note the last round, timestamp unchanged but sync much faster than previous sync from same timestamp. |
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.
Approved.
ARRR could be activated and send and swap functions works as expected. So users are not blocked and can use their funds
Need to additional enhancement or bug for activation starts on each app launch
ref: KomodoPlatform/komodo-defi-framework#1922
closes: https://github.com/KomodoPlatform/komodo-wallet-desktop/issues/2354
This PR introduces the
sync_params
feature in the newly released 1.0.7 API. This allows the user to specify which block to begin sync from - previously it was hardcoded as block 1900000 (for ARRR).After this PR is merged it will use the date selected by the user in settings, and where the setting does not exist (e.g. for zombie) it will use 2 days in the past
To test:
"first_sync_block":{"actual":1500000,"is_pre_sapling":false,"requested":1500000}
. You dont need to wait for full sync (it takes a long time), just confirm in logs.Note: Balance from transactions prior to the sync block will not be visible / included. Some sections are very, very slow.