Skip to content
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

AquaCell Failed Setup, will retry: 'Address' #121246

Closed
PaulAntonDeen opened this issue Jul 4, 2024 · 4 comments · Fixed by #121253
Closed

AquaCell Failed Setup, will retry: 'Address' #121246

PaulAntonDeen opened this issue Jul 4, 2024 · 4 comments · Fixed by #121253

Comments

@PaulAntonDeen
Copy link

PaulAntonDeen commented Jul 4, 2024

The problem

image

I connected the integration to my account but it will not load because of the 'address'. Sometimes on reload it triggers on the postcode, but i've been playing with how my adres is entered in the myaquacell app and now it triggers on the adres instead of the postcode. So what is the required format for the adres in the app?

What version of Home Assistant Core has the issue?

core-2024.7.0

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant OS

Integration causing the issue

AquaCell

Link to integration documentation on our website

https://www.home-assistant.io/integrations/aquacell

Diagnostics information

home-assistant_aquacell_2024-07-04T19-14-58.968Z.log

Example YAML snippet

No response

Anything in the logs that might be useful for us?

Logger: homeassistant.components.aquacell.coordinator
Source: helpers/update_coordinator.py:312
integration: Aquacell (documentation, issues)
First occurred: 21:09:53 (12 occurrences)
Last logged: 21:15:45

Unexpected error fetching Aquacell Coordinator data
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 312, in _async_refresh
    self.data = await self._async_update_data()
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/components/aquacell/coordinator.py", line 72, in _async_update_data
    softeners = await self.aquacell_api.get_all_softeners()
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/aioaquacell/aquacell_api.py", line 77, in get_all_softeners
    softeners.append(Softener(softener_as_json))
                     ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/aioaquacell/softener.py", line 25, in __init__
    self.location: Location = Location(data["location"])
                              ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/aioaquacell/softener.py", line 42, in __init__
    self.address: str = data["address"]
                        ~~~~^^^^^^^^^^^
KeyError: 'address'

Additional information

No response

@home-assistant
Copy link

home-assistant bot commented Jul 4, 2024

Hey there @Jordi1990, mind taking a look at this issue as it has been labeled with an integration (aquacell) you are listed as a code owner for? Thanks!

Code owner commands

Code owners of aquacell can trigger bot actions by commenting:

  • @home-assistant close Closes the issue.
  • @home-assistant rename Awesome new title Renames the issue.
  • @home-assistant reopen Reopen the issue.
  • @home-assistant unassign aquacell Removes the current integration label and assignees on the issue, add the integration domain after the command.
  • @home-assistant add-label needs-more-information Add a label (needs-more-information, problem in dependency, problem in custom component) to the issue.
  • @home-assistant remove-label needs-more-information Remove a label (needs-more-information, problem in dependency, problem in custom component) on the issue.

(message by CodeOwnersMention)


aquacell documentation
aquacell source
(message by IssueLinks)

@Jordi1990
Copy link
Contributor

@PaulAntonDeen Thanks for reporting! It seems like there's a problem when not all information is filled into the app. I'll look into it. Also the integration doesn't need this information anyway, so I will probably just remove the parsing of this data.

In my case all three fields, address, postcode and country are filled in the app and then it works, but I do see some strange response from the API when keeping one or multiple empty.

@Jordi1990
Copy link
Contributor

Jordi1990 commented Jul 4, 2024

Solved in 2024.7.1

@Jordi1990
Copy link
Contributor

@home-assistant close

@home-assistant home-assistant bot closed this as completed Jul 4, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Aug 3, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants