From 8a5bb688afe8ca44924c50d6573bfa9567a62189 Mon Sep 17 00:00:00 2001 From: Tyler Goodlet Date: Thu, 2 Oct 2025 20:08:56 -0400 Subject: [PATCH] binance: set `Pair.pegInstructionsAllowed = False` Lol, a cheeky unforeseen bug due to TOML's lack of a null type and thinking i can render an `Optional` field on a `msgspec.Struct` (defaulted to `None`) the `binance.symcache.toml` cache file.. I didn't catch this when i first updated to the 3.1 API in f7caa75228 because i never did a cache-files flush.. lesson learned and we **really need tests for this**!! --- piker/brokers/binance/venues.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/piker/brokers/binance/venues.py b/piker/brokers/binance/venues.py index 35388be9..23995c21 100644 --- a/piker/brokers/binance/venues.py +++ b/piker/brokers/binance/venues.py @@ -102,7 +102,7 @@ class Pair(Struct, frozen=True, kw_only=True): # https://developers.binance.com/docs/binance-spot-api-docs#2025-08-26 # will become non-optional 2025-08-28? # https://developers.binance.com/docs/binance-spot-api-docs#future-changes - pegInstructionsAllowed: bool|None = None + pegInstructionsAllowed: bool = False filters: dict[ str,