Skip to content

Add support for transactions with payees#14

Open
pmarciniak wants to merge 6 commits intosimon-v:masterfrom
pmarciniak:payees
Open

Add support for transactions with payees#14
pmarciniak wants to merge 6 commits intosimon-v:masterfrom
pmarciniak:payees

Conversation

@pmarciniak
Copy link
Contributor

This PR adds an option 'y'. If it's set, transaction entering flow will ask for a payee in addition to description. Entering payees has the same support for defaults and completion as entering descriptions.

Since either payee or description may now legitimately be empty, I've added a special value "-" to not accept the proposed default and enter an empty string instead.

Even with the 'y' option disabled, parsing of transactions is now aware of payees, so it no longer parses broken descriptions like Central Perk" "Scone.

@simon-v
Copy link
Owner

simon-v commented May 27, 2023

Why do you consider "descriptions" like Central Perk" "Scone broken? I've been using them to indicate payees for years.

@pmarciniak
Copy link
Contributor Author

Yeah, I guess you can use them to enter new transactions, but when you want to search for a transaction with a given description/payee, it becomes a bit more messy. Also, some payees may have multiple descriptions associated with them, and vice versa (e.g. transactions with "Groceries" description may exist for multiple payees), which makes the auto-completion feature work in a suboptimal way.

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