-
-
Notifications
You must be signed in to change notification settings - Fork 32.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Monzo integration #101731
Add Monzo integration #101731
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!
a061a4e
to
790fd08
Compare
Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍 |
There hasn't been any activity on this pull request recently. This pull request has been automatically marked as stale because of that and will be closed if no further activity occurs within 7 days. |
Thanks for the reviews and apologies for the delay making changes, moving house has kept me busy since new year. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is a merge conflict.
a8d6409
to
062d5e2
Compare
Also use list comprehensions for producing sensor lists
Apparently this wasn't useless This reverts commit c6b7109.
Adds support for joint/business/etc account pots
1588794
to
f3ca65c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks @JakeMartin-ICL 👍
Proposed change
This adds a new integration for Monzo, the UK bank. It provides balance and total balance sensors for each account and balance sensors for each pot. It also registers a webhook for transaction creation events which can be used as a device trigger, plus a simple service to transfer money between your own accounts and pots.
If you'd like to check out the integration on HACS, it can be found at: https://github.com/JakeMartin-ICL/ha-monzo/tree/main
The repo with the API wrapper is: https://github.com/JakeMartin-ICL/monzopy
Screenshots
Each account is shown in HA as a device
Each device has a balance sensor, plus a total balance if the account has pots
An example of an automation I use to keep my current account at a target I set, moving any excess to savings or pulling back into my current account if needed
Type of change
Additional information
Checklist
black --fast homeassistant tests
)*not 100% coverage
If user exposed functionality or configuration variables are added/changed:
If the code communicates with devices, web services, or third-party tools:
Updated and included derived files by running:
python3 -m script.hassfest
.requirements_all.txt
.Updated by running
python3 -m script.gen_requirements_all
..coveragerc
.To help with the load of incoming pull requests: