As pointed out by @macobo, some "include imported or not" logic is leaking into general query modifying functions and requires the developer to understand the logic behind it whenever doing operations such as remove_filters or set_property. This PR aims to avoid that, making it explicit and more close to the business logic where it's used.
I've also included comments in the relevant places and improved test coverage for timeseries.
Tests
[x] Automated tests have been added
Changelog
[x] This PR does not make a user-facing change
Documentation
[x] This change does not need a documentation update
Changes
As pointed out by @macobo, some "include imported or not" logic is leaking into general query modifying functions and requires the developer to understand the logic behind it whenever doing operations such as
remove_filters
orset_property
. This PR aims to avoid that, making it explicit and more close to the business logic where it's used.I've also included comments in the relevant places and improved test coverage for timeseries.
Tests
Changelog
Documentation
Dark mode