Skip to content

fix: Update stats for multi-level drill-down#62

Merged
wesm merged 1 commit intomainfrom
bug/drill-down-metrics
Dec 27, 2025
Merged

fix: Update stats for multi-level drill-down#62
wesm merged 1 commit intomainfrom
bug/drill-down-metrics

Conversation

@wesm
Copy link
Owner

@wesm wesm commented Dec 27, 2025

The transaction count and in/out totals in the top-right stats bar now correctly reflect all drill-down filters in multi-level navigation.

Previously, drilling from e.g. "Amazon > Groceries" would still show stats for all Amazon transactions. The bug was in get_filtered_df() which used an elif chain that only applied the first filter. Changed to sequential if statements to apply all dimension filters.

Also updated CLAUDE.md to explicitly prohibit git pull without permission.

🤖 Generated with Claude Code

The transaction count and in/out totals in the top-right stats bar now
correctly reflect all drill-down filters in multi-level navigation.

Previously, drilling from "Amazon > Groceries" would still show stats
for all Amazon transactions. The bug was in get_filtered_df() which used
an elif chain that only applied the first filter. Changed to sequential
if statements to apply all dimension filters.

Also updated CLAUDE.md to explicitly prohibit git pull without permission.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@wesm wesm merged commit 711780a into main Dec 27, 2025
8 checks passed
@wesm wesm deleted the bug/drill-down-metrics branch December 27, 2025 20:52
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.

1 participant