Repository navigation
remove obsolet prio settings - #4178
Merged
Merged
Conversation
LKuemmel
force-pushed
the
consumer_dev
branch
from
October 26, 2026 09:05
e8e84f1 to
82b4115
Compare
LKuemmel
changed the title
Contributor
There was a problem hiding this comment.
🟢 Approval recommended
The cleanup is narrowly scoped, includes migration regression coverage, and has no identified blocking defects.
No open issues 🎉
What changed in this PR
Removes obsolete priority settings from the charging data models and input handling, with a datastore migration to clean up existing configurations.
Changes:
- Removes legacy priority fields and their control-parameter validation.
- Adds datastore migration 155 to delete obsolete fields while preserving current load-management priorities.
- Adds cleanup and idempotency tests, preserves historical migration fixtures, and updates documentation.
| File | Description |
|---|---|
packages/helpermodules/update_config.py |
Adds migration 155 to remove obsolete priority fields. |
packages/helpermodules/update_config_test.py |
Tests cleanup and idempotency; preserves historical priority payloads. |
packages/helpermodules/setdata.py |
Removes validation for the obsolete priority parameter. |
packages/control/ev/ev.py |
Updates the module docstring to omit priority from control parameters. |
packages/control/ev/charge_template.py |
Removes the obsolete prio model field. |
packages/control/chargepoint/chargepoint_data.py |
Removes the obsolete connected-vehicle priority field. |
Review effort: Balanced
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.