Skip to content

Conversation

@justin-jiajia
Copy link
Contributor

I done this by retrieving a summary of the same period without any filters. Maybe there's a more efficient way that I didn't come up with.

@justin-jiajia justin-jiajia marked this pull request as draft December 22, 2025 14:54
@muety
Copy link
Owner

muety commented Dec 25, 2025

While this is not optimal performance-wise since we're now loading two summaries for every filtered request, it's still the easiest and most straightforward way I can think of rn without employing a crazy lot of new changes. Should be fine. Thanks for the contribution! 👍

@muety muety merged commit 348ca55 into muety:master Dec 25, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants