Documentation for Sure Pet is not complete

[i00]

• Home Assistant release with the issue: 0.104.3

• Operating environment (Hass.io/Docker/Windows/etc.): Docker

• Integration causing this issue: Sure Petcare

• Link to integration documentation on our website: Sure Petcare

Additional information

The documentation just states:

For now, please use the sp_cli.py from @rcastberg to fetch the IDs from the Sure Petcare API. With default setting, the IDs will be written as JSON to ~/.surepet.cache

This method doesn't even work.. It failed to obtain a mac address and crashed - so I changed the getmac() function to return my mac. Then it bombed out while getting json data... the surepet.cache file was somewhat readable but too corrupted to get useful information out of.

Rather than just fix the documentation, this lib should be built into the HA integration ... my advice would be to put the username / password in the config and have an entry with an attribute (rather than state so it is not logged) that contains all of the pets, flaps etc.

[frenck]

Please open up issues for our documentation here: https://github.com/home-assistant/home-assistant.io/issues/new/choose

Secondly, could you please fill out the issue template, instead of removing large chunks of it? Thanks! 👍

[probot-home-assistant[bot]]

Hey there @benleb, mind taking a look at this issue as its been labeled with a integration (surepetcare) you are listed as a codeowner for? Thanks!

[benleb]

Feel free to make a PR @i00... it worked for many other users by the way. I will focus on the implementing more info/attributes available via the API first

Theres another, probably easier way to get the IDs, I will update the docs soon :)

[springstan]

Closing this issue as this is a documentation issue.