sampsyo / wideq

reverse-engineered client for the LG SmartThinQ API
https://pypi.org/project/wideq/
MIT License
333 stars 158 forks source link

Added a check when performing a post. #102

Open beele opened 4 years ago

beele commented 4 years ago

If a device control was performed, release the control back. Also refactored the logic a bit so the both the api_root and path are passed to the post method

beele commented 4 years ago

80 chars FFS. People have widescreen monitors these days... Not saying lines can be 200 long but 80 is like WTF

ncovercash commented 4 years ago

Missed a line! 🙃

For real though, 100 or 120 if any at all is much saner these days

ncovercash commented 4 years ago

From my experience, a delControlPermission is needed for every control request sent, otherwise the oofficial mobile app locks out for a while

sampsyo commented 4 years ago

Sure, makes sense—but let's put it where we initiate those rtiControl control calls to lgedm_post rather than inside the function based on an if to see what type of request it is.

ncovercash commented 4 years ago

Understandable!