Skip to content
This repository has been archived by the owner on Apr 3, 2024. It is now read-only.

Migrating and merging this component with the Ecovacs core integration #512

Open
edenhaus opened this issue Jan 24, 2024 · 20 comments
Open

Comments

@edenhaus
Copy link
Contributor

edenhaus commented Jan 24, 2024

I'm currently migrating this integration to the core to make it easier to use it and add the ability to translate it into all supported languages.
This issue is here to track the status.

This whole component will be included in HA 2024.2 except for the following things:

  • The events cleaning_job and custom_command.
  • The last cleaning and stats type sensors

Migrations steps

1: Remove all Deebot 4 Home Assistant config entries
2: Uninstall Deebot 4 Home Assistant via HACS
3: Restart Home Assistant
4: Setup the builtin Ecovacs integration

If you have issues setting up the core integration, make sure no deebot folder exists under [config dir]/custom_components (Delete it if it exists) and restart Home Assistant

@edenhaus edenhaus pinned this issue Jan 24, 2024
@scooper1

This comment was marked as outdated.

@edenhaus

This comment was marked as outdated.

@Ali229

This comment was marked as outdated.

@MVladislav

This comment was marked as outdated.

@mariomaz87

This comment was marked as outdated.

@edenhaus

This comment was marked as outdated.

@jayscovill

This comment was marked as outdated.

@edenhaus

This comment was marked as outdated.

@aidbish
Copy link

aidbish commented Feb 17, 2024

I have migrated to the core integration, but it seems it no longer has the command to refresh, will this be added at a later date?

@edenhaus
Copy link
Contributor Author

I have migrated to the core integration, but it seems it no longer has the command to refresh, will this be added at a later date?

Nope as also in this integration that command is deprecated. Please use the builtin update_entity service

@Stopka
Copy link

Stopka commented Feb 28, 2024

I have migrated to the core integration and nowI am missing the last_cleaning sensor. Especially I used it's timestamp attribute in my automations to check if cleaning was already done today.
Why is it missing? Or is there other way to get this value?

@myhomeiot
Copy link

Also missing way to get last_cleaning, meanwhile I make this by saving time at input_text but sure this was a good sensor in previous version.

@jayscovill
Copy link

jayscovill commented Mar 1, 2024

I've never had a last cleaning sensor for the X1 Omni and I've done the same thing. As part of the cleaning automation I write the current date/time to a input_datetime.

service: input_datetime.set_datetime
target:
  entity_id: input_datetime.vacuum_cleaning_start
data:
  datetime: "{{ now().strftime('%Y-%m-%d %H:%M:%S') }}"

@ianackerley
Copy link

I feel like I'm missing something really obvious here (sorry) but without the cleaning_job event what's the best way to monitor for successful completion of a job?

@edenhaus
Copy link
Contributor Author

edenhaus commented Mar 2, 2024

I have migrated to the core integration and nowI am missing the last_cleaning sensor. Especially I used it's timestamp attribute in my automations to check if cleaning was already done today.
Why is it missing? Or is there other way to get this value?

You can use the state of the vacuum itself. You can use it like in the comment above or you can extract the value with https://www.home-assistant.io/integrations/history_stats/

The last cleaning sensor was not migrated as all information are already available from other sensors

@edenhaus
Copy link
Contributor Author

edenhaus commented Mar 4, 2024

I feel like I'm missing something really obvious here (sorry) but without the cleaning_job event what's the best way to monitor for successful completion of a job?

Good point I completely forgot about this event. Sorry. Will check with the core team, if there is a better solution as firing the event

@smarthomesydney
Copy link

Hi @edenhaus
Sorry to see you merge. Have used your integration for years. Thank you

Quick Q
Without the custom_command, how do I call only specific rooms to be cleaned, based on the room number in the entity info?

@Stopka
Copy link

Stopka commented Mar 6, 2024

I have migrated to the core integration and nowI am missing the last_cleaning sensor. Especially I used it's timestamp attribute in my automations to check if cleaning was already done today. Why is it missing? Or is there other way to get this value?

Ok, I found solution that serves me well. I created event-triggered template sensor that holds the needed information for me:

template:
  - trigger:
      - platform: state
        entity_id: 
          - vacuum.r2d2
        to: 
          - cleaning
    sensor:
      - name: R2D2 Last cleaning time
        state: "{{ now() }}"
        device_class: timestamp
        icon: 'mdi:clock' 

@edenhaus
Copy link
Contributor Author

edenhaus commented Mar 19, 2024

I feel like I'm missing something really obvious here (sorry) but without the cleaning_job event what's the best way to monitor for successful completion of a job?

I will implement event entities for it. Should be available hopefully with the next version :)
home-assistant/core#113826

@edenhaus
Copy link
Contributor Author

Sorry to see you merge. Have used your integration for years. Thank you

Why sorry? Having the integration directly in the core makes it easier to maintain it.

Without the custom_command, how do I call only specific rooms to be cleaned, based on the room number in the entity info?

Why do you need a custom component for it? Why aren't you using the send_command for it?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests