Skip to content

Show transactions from all accounts by default, and add hiddenByAccount to returned fields#170

Open
grablair wants to merge 6 commits intohammem:mainfrom
grablair:main
Open

Show transactions from all accounts by default, and add hiddenByAccount to returned fields#170
grablair wants to merge 6 commits intohammem:mainfrom
grablair:main

Conversation

@grablair
Copy link
Collaborator

fixes #168

A few months ago, Monarch changes the "Hidden from budgets and reports" into "Hidden from transaction list, budgets, and reports", which is a significant behavior change.

This modifies the transaction list API call to list all transactions from all accounts, but also add a hiddenByAccount field to the returned accounts, so that information is still available.

This is on by defaults, but can be turned off by a new parameter on the get_transactions method.

@grablair grablair requested review from Copilot and hammem October 31, 2025 21:27
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds support for filtering transactions based on their visibility, specifically including or excluding transactions from accounts hidden from lists, reports, and budgets.

  • Adds a new show_all_transactions parameter to the get_transactions method with a default value of True
  • Adds the hiddenByAccount field to the GraphQL query fragment to retrieve this property for each transaction
  • Implements the filtering logic to set transactionVisibility filter to "all_transactions" when the parameter is True

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@alex-gable
Copy link

Pretty strange you've updated authorship and the pypi link @grablair. Was that intentional?

@tmas
Copy link

tmas commented Jan 16, 2026

@alex-gable @hammem hasn't committed to the repo for almost a year now, so that may have been an effort to keep releases flowing. If this repo remains inactive, it may be worth forking and moving community efforts to the fork given that it seems like the current version may not actually work

ETA: that being said, the diff contains a reference to a repo that doesn't actually exist (hammem/monarchmoney-grablair)

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.

Feature/Fix: Add transactionVisibility parameter to GetTransactionList; default to all_transactions

3 participants