MoneyTools / MyMoney.Net

MyMoney is a rich client .NET ClickOnce application for managing your personal finances. It is written entirely in C# and is designed for programmers who want easy access to their data and who want to quickly and easily add their own features. Your data will not be locked up in some proprietary format, it is yours to do with as you like.
https://moneytools.github.io/MyMoney.Net/
66 stars 21 forks source link

F12 Find and create Transfer transaction #78

Closed jpdup closed 1 year ago

jpdup commented 1 year ago

Use F12 to find matching transactions in other accounts. If match found then offer to the user to transform the two disconnected transactions into a "Transfer" transaction.

andreva7 commented 1 year ago

Very cool!

From: jpdup @.> Sent: October 31, 2022 10:53 AM To: clovett/MyMoney.Net @.> Cc: Subscribed @.***> Subject: [clovett/MyMoney.Net] F12 Find and create Transfer transaction (PR #78)

Use F12 to find matching transactions in other accounts. If match found then offer to the user to transform the two disconnected transactions into a "Transfer" transaction.


You can view, comment on, or merge this pull request online at:

https://github.com/clovett/MyMoney.Net/pull/78https://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fclovett%2FMyMoney.Net%2Fpull%2F78&data=05%7C01%7C%7Cd06f75defb1a4aeae4f908dabb68bedf%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C638028355799867386%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=J3l8RqWh3Ol1O8cngckpP6quaBI%2FCwNhNoFgXRti5p4%3D&reserved=0

Commit Summary

File Changes

(1 filehttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fclovett%2FMyMoney.Net%2Fpull%2F78%2Ffiles&data=05%7C01%7C%7Cd06f75defb1a4aeae4f908dabb68bedf%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C638028355799917165%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=2idUDxW73NJjbtR7e%2BW0LNML%2BHtnMtkXFGUBEuBwi5A%3D&reserved=0)

Patch Links:

- Reply to this email directly, view it on GitHubhttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fclovett%2FMyMoney.Net%2Fpull%2F78&data=05%7C01%7C%7Cd06f75defb1a4aeae4f908dabb68bedf%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C638028355799976900%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=kQCxaqfCp4fzUJO%2BQE3yT73OFXHnG4IvnjJApo0nt4g%3D&reserved=0, or unsubscribehttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAJBV66FQE5CFKOC2DHUSDN3WGABPTANCNFSM6AAAAAARTKFJ6A&data=05%7C01%7C%7Cd06f75defb1a4aeae4f908dabb68bedf%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C638028355799986855%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=NybLMiMMuqrJw4XnYVORdAuC2TAaZlc5oo7vcwUdhcc%3D&reserved=0. You are receiving this because you are subscribed to this thread.Message ID: @.***>

clovett commented 1 year ago

Super tight code, love how you reused the Query system! Thanks, JP! I'll take this as is, and we can brainstorm on whether we want to tackle a UX for selection from multiple transactions with a custom dialog. I think if there are multiple hits we can also leverage the auto-categorization feature to find the most likely match based on previous transfers in the same account.