This commit adds UR functions for UR tasks,
I believe that the right place to get
UR strings is the wallet code itself,
especially because it allows us to
skip the part when we have to store
things to file to encode them later.
Now we are fully in memory
Things broken in the commit
- ledger support.
AUTO_LOCK_CMD macro causes compile time
issues with this patch. I don't know why
just yet, this is a issue that I'll fix
later. However (considering the purpose
of this patch) it is not a dealbreaker.
Monero wallet crashes (sometimes) when it is syncing,
while the proper solution (that can be seen in feather)
is to not store wallet while it is being synced, this is not
acceptable for mobile wallets where OS can just come
and kill the wallet because it felt like it.
This patch depends on the background-sync patch, but
to use it as a standalone fix grabbing the definition for the
LOCK_REFRESH macro should be enough.
tobtoht suggested:
_say you want to store every 15 minutes during background sync. you stop the refresh every 15 minutes. then do something like this in the callback:_
```
// Make sure this doesn't run in the refresh thread
onRefreshed() {
if (hasItBeen15MinutesSinceWeStored()) {
store();
}
if (shouldWeContinueRefreshing()) {
startRefresh();
}
}
```
which works for crashes after the wallet is initially synced
but doesn't solve the issue for wallet that are syncing (it
would just wait for it to finish before actually storing).
Also imo store() functin should store the wallet, no matter
the current state.
* bumped version to v1.0.0
* added fast sync points up to 325,000
* added fast sync points up to 325,000 (part 2)
* RPC address formatting fixes
---------
Co-authored-by: Some Random Crypto Guy <somerandomcryptoguy@protonmail.com>
* bumped version to v1.0.0
* added fast sync points up to 325,000
* added fast sync points up to 325,000 (part 2)
* initial import of CLI wallet improvements for address printing
* updated wallet address display in various places
---------
Co-authored-by: Some Random Crypto Guy <somerandomcryptoguy@protonmail.com>
* removed HF11
* fixed problem with AUDIT generation; fixed issue with empty PROTOCOL TX after HF10
* fixed protocol_tx version if empty at HF10 commencement; added validation of correct address type for destinations on TRANSFER
* fix is_carrot for stake txs
---------
Co-authored-by: Some Random Crypto Guy <somerandomcryptoguy@protonmail.com>
Co-authored-by: auruya <dream.glorix@gmail.com>
Czarek Nakamoto