-
-
Notifications
You must be signed in to change notification settings - Fork 89
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
Rework config flow #1501
Rework config flow #1501
Conversation
fix config migration when keys missing
add minor version
…6wxjx9j3pwp4 Allow multiple chargers to connect
…6wxjx9j3pwp4 switch to central services
Caution Review failedThe pull request is closed. WalkthroughThe pull request introduces a comprehensive update to the OCPP (Open Charge Point Protocol) Home Assistant integration, focusing on multi-charger support and configuration management. The changes span multiple files, introducing a new configuration structure that allows multiple charge points, enhancing configuration flexibility, and improving overall component robustness. Key modifications include adding new configuration constants, restructuring settings management, and updating test suites to reflect these changes. Changes
Sequence DiagramsequenceDiagram
participant User
participant ConfigFlow
participant CentralSystem
participant ChargePoint
User->>ConfigFlow: Start Configuration
ConfigFlow->>ConfigFlow: Validate Central System Settings
ConfigFlow->>ConfigFlow: Configure Websocket Parameters
ConfigFlow->>ConfigFlow: Add Charge Points
ConfigFlow->>CentralSystem: Create Central System
CentralSystem->>ChargePoint: Initialize Charge Points
ChargePoint->>ChargePoint: Configure Settings
ChargePoint-->>CentralSystem: Register Charge Points
CentralSystem-->>User: Configuration Complete
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (26)
✨ Finishing Touches
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1501 +/- ##
==========================================
- Coverage 94.69% 91.56% -3.14%
==========================================
Files 12 12
Lines 1943 2028 +85
==========================================
+ Hits 1840 1857 +17
- Misses 103 171 +68 ☔ View full report in Codecov by Sentry. |
Summary by CodeRabbit
Based on the comprehensive summary of changes, here are the release notes:
Release Notes v0.8.0
New Features
Configuration Changes
Performance Improvements
Bug Fixes
Compatibility