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 OurGroceries integration #103387
Add OurGroceries integration #103387
Conversation
Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍 |
5ebcfbe
to
cdd2456
Compare
8565af8
to
a253c5e
Compare
a253c5e
to
e0aa5e3
Compare
@joostlek are there any additionally required changes? |
@joostlek is there anything still missing here? I'd love to get this merged in time for the beta |
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.
Final remarks
e0aa5e3
to
e8aa27b
Compare
The test failures are in different integrations and don't seem related to this PR |
e8aa27b
to
fc5f895
Compare
After a rebase (no additional changes) the tests pass. Are we good to get this merged? |
I'll take another look when I get home. |
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.
Please address the comments in a new PR. Thanks!
Thanks @MartinHjelmare. I addressed those in #104606. |
Proposed change
Add an integration with OurGroceries and expose shopping lists as Todo entities.
Type of change
Additional information
Checklist
black --fast homeassistant tests
)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: