From eb49e163510d589e3397e57100c9cf7d72d9dd2c Mon Sep 17 00:00:00 2001 From: epenet <6771947+epenet@users.noreply.github.com> Date: Thu, 21 Nov 2024 19:18:25 +0000 Subject: [PATCH] Update comment --- homeassistant/config_entries.py | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/homeassistant/config_entries.py b/homeassistant/config_entries.py index 38474543a627c..c532638d45bad 100644 --- a/homeassistant/config_entries.py +++ b/homeassistant/config_entries.py @@ -1487,8 +1487,12 @@ async def async_finish_flow( ) if existing_entry is not None and flow.handler != "mobile_app": - # This causes the old entry to be removed and replaced when it - # should most likely update the previous entry and abort the flow + # This causes the old entry to be removed and replaced, when the flow + # should instead be aborted. + # In case of manual flows, integrations should implement options, reauth, + # reconfigure to allow the user to change settings. + # In case of non user visible flows, the integration should optionally + # update the existing entry before aborting. # see https://developers.home-assistant.io/blog/2024/11/22/config-flow-unique-id/ report_usage( "creates a config entry when another entry with the same unique ID "