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

Local mode #28

Closed
Phype opened this issue Oct 27, 2021 · 1 comment
Closed

Local mode #28

Phype opened this issue Oct 27, 2021 · 1 comment

Comments

@Phype
Copy link
Contributor

Phype commented Oct 27, 2021

I just tested something: When you turn the robot around, you can set the localpw for the local connection without breaking the connection to the cloud. This means that it is pretty easy to make use of the local mode of purei9_unofficial. Users have just turn the robot around and run a command (Maybe this cloud even be included in the home assistant intergration).

As the official app isn't using the local interface anymore anyway, i don't think it breaks anything if we set the localpw to another value as it was before.

So i though maybe it makes sense to kind of include this in the integration. Something like supporting both modes of controlling the robot, depending on what it currently available or something.

What do you think?

@Ekman
Copy link
Owner

Ekman commented Nov 13, 2021

Thanks for the suggestion, I appreciate it!

I'd suggest you talk to the guys over at anhaehne/homeassistant_electrolux_purei9 regarding this. They've created a Home Assistant library for offline Purei9.

This change would either mean a breaking change in the API facing the users of this library or a configuration value, both which I'm not entirely fond of. When I create software I tend to keep them as simple as possible with just the bare necessary configuration. It's easy to create a "one-solution-fits-all" which is not very fun to maintain. In this case, we have a separate integration that could benefit from this instead.

I have so much more things that I'd rather focus on regarding this library.

@Ekman Ekman closed this as completed Nov 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants