Skip to content

Add stop charge button to renault integration #88003

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.

Sign up for GitHub

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

Merged
epenet merged 11 commits into from
Feb 28, 2023
Merged

Add stop charge button to renault integration #88003

Show file tree
Hide file tree
Changes from 4 commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
84e5b73
Added service to start/stop charge
rodriguestiago0 Feb 13, 2023
561eac0
Remove comment
rodriguestiago0 Feb 13, 2023
473fc96
Fixed service
rodriguestiago0 Feb 13, 2023
74d08ec
removed service for start/stop charge
rodriguestiago0 Feb 13, 2023
142fd14
Remove version
rodriguestiago0 Feb 13, 2023
c1252d6
Format
rodriguestiago0 Feb 13, 2023
3d82028
Revert change
rodriguestiago0 Feb 13, 2023
ed896fb
Fix lint
epenet Feb 14, 2023
86aff2d
Add tests
epenet Feb 14, 2023
b2cc6ab
Merge branch 'dev' into add-stop-charge-renault
rodriguestiago0 Feb 15, 2023
a880a3f
Merge branch 'dev' into add-stop-charge-renault
rodriguestiago0 Feb 15, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions homeassistant/components/renault/button.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -70,5 +70,11 @@ async def async_press(self) -> None:
icon="mdi:ev-station",
name="Start charge",
requires_electricity=True,
),RenaultButtonEntityDescription(
Comment thread
rodriguestiago0 marked this conversation as resolved.
Outdated
async_press=lambda x: x.vehicle.set_charge_stop(),

@epenet epenet Feb 13, 2023

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Have you tested this feature?

Feedback on the renault-api library has been inconsistent, and the function doesn't appear to work on all vehicles (does it actually work on yours?)

@rodriguestiago0 rodriguestiago0 Feb 13, 2023

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The service worked on my Dacia Spring. I will test the button today.

@rodriguestiago0 rodriguestiago0 Feb 13, 2023

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just tested it and worked with my car.

Dacia spring.

@epenet epenet Feb 13, 2023

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great can you please adjust tests accordingly?

@rodriguestiago0 rodriguestiago0 Feb 13, 2023

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we split the renault and dacia integration? For example dacia spring don't allow schedule charge?

Is possible to have a base integration that is common to both and then specific for each brand?

@epenet epenet Feb 13, 2023

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is the same library so I think it's better to have a single integration.
Even some Renault vehicles don't support schedule charge...
It is better to adjust documentation for this

key="stop_charge",
icon="mdi:ev-station",
name="Stop charge",
requires_electricity=True,
),
)
3 changes: 2 additions & 1 deletion homeassistant/components/renault/manifest.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,5 +8,6 @@
"iot_class": "cloud_polling",
"loggers": ["renault_api"],
"quality_scale": "platinum",
"requirements": ["renault-api==0.1.12"]
"requirements": ["renault-api==0.1.12"],
"version": "0.1"
Comment thread
rodriguestiago0 marked this conversation as resolved.
Outdated
}
5 changes: 5 additions & 0 deletions homeassistant/components/renault/renault_vehicle.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -151,6 +151,11 @@ async def set_charge_start(self) -> models.KamereonVehicleChargingStartActionDat
"""Start vehicle charge."""
return await self._vehicle.set_charge_start()

@with_error_wrapping
async def set_charge_stop(self) -> models.KamereonVehicleChargingStartActionData:
"""Stop vehicle charge."""
return await self._vehicle.set_charge_stop()

@with_error_wrapping
async def set_ac_stop(self) -> models.KamereonVehicleHvacStartActionData:
"""Stop vehicle ac."""
Expand Down
13 changes: 1 addition & 12 deletions homeassistant/components/renault/services.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -74,15 +74,4 @@ charge_set_schedules:
]
required: true
selector:
object:

charge_start:
description: Start charge on vehicle.
fields:
vehicle:
name: Vehicle
description: The vehicle to send the command to.
required: true
selector:
device:
integration: renault
object:
Comment thread
epenet marked this conversation as resolved.
Outdated