-
r4v3r23[m]
jberman: any progress on background sync?
-
jberman[m]
ETA 1-2 weeks. I'm prioritizing fixing 8566 atm. I figure bug fixing takes priority over new features. Hoping to be done with that today or tomorrow
-
jberman[m]
While working on 8566 I also started thinking about a different feature that gingeropolous has brought up in the past: allowing a user to "skip" scanning to current height
-
jberman[m]
So if you haven't used your wallet in a while and know you haven't sent/received any txs you can just skip scanning. Seems like it would be simple to implement reusing what I'm doing to fix 8566, and imo would be more impactful than background sync
-
hyc
I suspect people will use that option all the time, and eventually miss a received txn and complain
-
jberman[m]
I was thinking something like: in advanced mode, there's a checkbox in the "Interface" tab that lets you enable the feature. If the feature is enabled, while the wallet is syncing it would display a "skip" button above "Wallet blocks remaining". When you enable the feature, a modal warns you of the consequences before you confirm. Then when you hit the "skip" button, a modal warns you of the consequences every time but you can click a
-
jberman[m]
checkbox that says "don't show this again"
-
jberman[m]
in the GUI
-
r4v3r23[m]
i agree with hyc, seems like a feature that can easily cause headaches
-
r4v3r23[m]
background sync is much simpler and effective
-
Lyza
<jberman[m]> feather wallet is working on this alongside with an option to let you import individual transactions by TX ID
-
Lyza
so if you know specifically what transactions you received, you can skip and add them manually, with the added bonus of maybe helping people with missing transaction issues more generally
-
Lyza
by feather wallet I mean tobtoht of course :D
-
Lyza
what's background sync though
-
r4v3r23[m]
Lyza: allowing wallet to sync in the background while keeping only the viewkey hot
-
r4v3r23[m]
spendkey decrypts when user enters password, and grabs all the synced blocks
-
r4v3r23[m]
major UX improvement for mobile wallets
-
Lyza
ok I see. yeah monerujo already does this but I guess it must keep the spend key open
-
r4v3r23[m]
monerujo just keeps the app open in background
-
r4v3r23[m]
jberman's feature will be more secure
-
Lyza
yeah it sounds like it -- but you'd need to keep the app open anyway right? I was thinking on desktop this + a system tray icon would be nice. don't think it replaces the skip sync feature though, it seems kinda nice especially if you can also import individual TX IDs
-
r4v3r23[m]
yes but it can be locked in the background, and just have an android service running
-
Lyza
def an improvement
-
sgp[m]
I'm trying to restore a wallet from monero-wallet-cli, and it's ignoring the restore height command when passed as `restore height <block>` and in the json file `"restore_height":"<block>"`. Does anyone else have this problem?
-
sgp[m]
I run -monero-wallet-cli, and then it prompts for a restore height. Luckily it properly accepts `"setup-background-mining":"0"`
-
sgp[m]
s/`/`--/, s//-/
-
ofrnxmr[m]
Whats the full command?
-
ofrnxmr[m]
`monero-wallet-cli --daemon-address <node> --restore-from-seed --restore-date <block>`
-
ofrnxmr[m]
I can try to reproduce
-
selsta
sgp[m]: the command is `--restore-height` and not `restore height`
-
selsta
at least I'm not aware of such a command
-
ofrnxmr[m]
I have no issue with
-
ofrnxmr[m]
`./monero-wallet-cli --daemon-address 192.168.0.12:18089 --restore-from-seed --restore-height 2000000`
-
ofrnxmr[m]
It starts at 0 and speedruns 10k blocks at a time until 2000000, then starts scanning
-
sgp[m]
I'll keep looking, but for some reason I can't get it to work with --restore-height. Thanks for checking
-
selsta
what exactly doesn't work?
-
plowsof
in the json file i use "scan_from_height":
-
selsta
yep, with json file it seems to be `scan_from_height`
monero.stackexchange.com/a/11045