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

Release v4.0.0 #113

Draft
wants to merge 27 commits into
base: master
Choose a base branch
from
Draft

Release v4.0.0 #113

wants to merge 27 commits into from

Conversation

skydiver
Copy link
Owner

@skydiver skydiver commented Oct 21, 2020

Goals:

  • switch to API v2
  • throw exceptions on errors
  • update dependencies

v2 current status

method using v2 throws exception
checkDeviceUpdate
checkDevicesUpdates
deviceControl
getCredentials
getDevice
getDeviceChannelCount
getDeviceCurrentTH
getDeviceCurrentTemperature
getDeviceCurrentHumidity
getDeviceIP
getDevicePowerState
getDevicePowerUsage
getDevicePowerUsageRaw
getDevices
getFirmwareVersion
getRegion
makeRequest
openWebSocket
saveDevicesCache
setDevicePowerState
toggleDevice

Pending

  • update docs: toggleDevice rename to toggleDevicePowerState
  • update docs: checkDevicesUpdates response has changed

@skydiver skydiver force-pushed the release/4.0.0 branch 3 times, most recently from 3cd9b98 to c302c62 Compare October 21, 2020 22:53
@skydiver skydiver marked this pull request as draft October 21, 2020 23:06
@maxbeech
Copy link

Hey @skydiver! This looks really awesome. Any idea on ETA please?

@pierred510
Copy link

HI
Is there someone still working on this release ?
Possible to fix this error ? :
#65

@alexose
Copy link

alexose commented Jun 10, 2022

Bump. Would be great to get this across the finish line.

@vmagalhaes
Copy link

HI Is there someone still working on this release ? Possible to fix this error ? : #65

Will be nice if could fix this issue, it's being a real problem.

@johnjmikucki
Copy link

Agreed, I'd be happy to help think up tests or author docs but I have exactly zero javascript. 😅

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

Successfully merging this pull request may close these issues.

None yet

6 participants