Add breaking changes warning about blueprint#288
Add breaking changes warning about blueprint#288sveinse merged 1 commit intocustom-components:masterfrom
Conversation
sveinse
left a comment
There was a problem hiding this comment.
This looks good. Added one small comment.
| The changes to the Charger mode will also cause the | ||
| [Load Balancing blueprint](#load-balancing-your-charger) compatible with 0.7.x | ||
| to stop working. There is an [open PR](https://github.com/svenakela/ha/pull/10) | ||
| for a 0.8.x-compatible version. If you use this blueprint, or a variant of it, | ||
| you will need to update your blueprint/automation accordingly. |
There was a problem hiding this comment.
Perhaps add an information box that the blueprint is language dependent, and that it might need modification if not using English?
There was a problem hiding this comment.
I think that belongs in the
blueprint readme, not in this repo. We already refer to that for "How to set up the automation, how the logic works and what all settings mean".
There was a problem hiding this comment.
Yes, I agree. However, it seems we're getting the issues about it, right?
There was a problem hiding this comment.
Well in this case we were the ones who broke it, so that's fair imo.
On the other hand, I see we have a blueprint issue in #179, but again, I think that belongs in the svenakela/ha-repo, not here. If there's anything that needs to be done inside the integration to fix it, then sure, but the issue with #179 is purely from the setup of the blueprint.
No description provided.