Skip to content

currency converter tool#113

Open
angadsingh wants to merge 1 commit intohammem:mainfrom
angadsingh:main
Open

currency converter tool#113
angadsingh wants to merge 1 commit intohammem:mainfrom
angadsingh:main

Conversation

@angadsingh
Copy link

@angadsingh angadsingh commented Jul 30, 2024

converts transactions and account balance history between currencies (default: USD to CAD).

it is meant to be run periodically. will resume from its last state. state is saved inside MM itself. transactions are tagged (e.g. a tag like USD2CAD - you create the tag). the original amount is put in the "notes" of the transaction in case you want to rollback.

account balance history is also checkpointed (creates a special checkpoint transaction with $0 balance - just to keep the last conversation date).

created this to use MM in canada.

@jeeftor
Copy link

jeeftor commented Aug 16, 2024

Have you figured out how to determine what currency an account is using? I can do it with SimpleFin which wraps mx.com - but haven't seen it in monarch

@angadsingh
Copy link
Author

that might be an overkill for this since it runs as a script anyway, we can just have the user fill a dict with the currencies of each account they want conversions on.

Copy link
Owner

@hammem hammem left a comment

Choose a reason for hiding this comment

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

@angadsingh , I appreciate you adding this, but I don't think this belongs in the library. It will add multiple dependencies to the build, processing of zip files on disk, and other complexities that may be a lot to keep functional or introduce errors on end-user environments that are challenging to debug.

If folks frequently need currency conversion, it's worth considering, but this doesn't seem central to having a Python API for Monarch.

@jeeftor
Copy link

jeeftor commented Jan 19, 2025 via email

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.

3 participants