Fix second condition to check for wallet_id
The second condition for the OR statement was the same as the first. It should be verifying that a wallet_id was passed in.
* disallow empty SpendBundles from being added to the mempool
* Update chia/full_node/mempool_manager.py
Co-authored-by: Adam Kelly <338792+aqk@users.noreply.github.com>
Co-authored-by: Adam Kelly <338792+aqk@users.noreply.github.com>
* When looking for peers, try all DNS servers first, then try the introducer. Start from a random dns server each restart.
* Simplify randomizing dns server order
Includes:
Fix for NFT metadata not displaying properly
Optimizations for displaying large NFT galleries
Added an button to clear the NFT cache in Settings --> NFT
NFT gallery now remembers the scroll position when navigating between
the gallery and NFT details
Fixes to the NFT gallery searching/filtering capabilities
Fixed an issue where text field component did not recognize rules
Without this the representation of the resulting methods look like `<function api_request.<locals>.inner.<locals>.wrapper at 0x7ff7ff655990>`. With, they are identifiable as to the method they wrap such as `<function FullNodeAPI.request_peers at 0x7fcc85d0d990>`.
* Use Chia-Network/actions/check-commit-signing
* @add_check_commit_signing
* use @main
* move check commit signing to a dedicated workflow
* Update check-commit-signing.yml
* fix fee ammt
* remove unit type
* fix fee calc
* Revert "Merge commit '30b9173eb205db5476b72bba8ef61dd2c3d34645' into checkpoint/main_from_release_1.6.2_30b9173eb205db5476b72bba8ef61dd2c3d34645"
This reverts commit 9884a729ac, reversing
changes made to 39d733599d.
* clear up cli args
* Update coins.py
* fix lint
* add seperate error for only 1 coin with combine.
* make daemon heartbeat configurable and increase default
* Fix up daemon rpc test
* Fix dumb error with parameters
* Restore formatting
* Various updates from feedback
* Update tests/core/test_daemon_rpc.py
use config value for heartbeat
Co-authored-by: Kyle Altendorf <sda@fstab.net>
* black fixes
Co-authored-by: Kyle Altendorf <sda@fstab.net>
* Simplify puzzle hash subscribing
* Simplify coin id subscribing
* Use the result of `chunks` in the loop definition
* Calculate the duration in place
* Filter before split to chunks and update the already checked at the end
* Adjust empty set check
* wallet: Don't add derived puzzle hashes to the `WalletInterestedStore`
We already use the derived puzzle hashes when we subscribe to the node
here
39d733599d/chia/wallet/wallet_node.py (L1291).
I assume this was done to make sure we subscribe to new puzzle hashes
but at the end it leads to redundant entries in `derivation_paths` vs
`interested_puzzle_hashes` so we can just subscribe here instead? If we
want to keep them in both tables for some reason we may at least get rid
of `get_puzzle_hashes_to_subscribe` and use the ones from the interest
store only and maybe adjust the other related methods?
* See if there are duplicates in tests
* Add a height timestamp for wallet notifications
* import sqlite3
* lint
* isort
* catch specific error
* Add a test for db backcompat
* Add a test for correct height
* mypy