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

Garage door opener support MSG100 #4

Merged
merged 4 commits into from
Feb 3, 2021
Merged

Garage door opener support MSG100 #4

merged 4 commits into from
Feb 3, 2021

Conversation

STDevMan
Copy link
Contributor

@STDevMan STDevMan commented Feb 2, 2021

I'll test this out a bit more, but it works so far. The UUID might only be needed on the meross main settings, not sure yet

added msg100 garage door support
added support for msg100 garage door
@dehsgr
Copy link
Owner

dehsgr commented Feb 2, 2021

@STDevMan very cool! Thanks for contributing. I'll suspend merge until

  • you finalized your tests and get are more safe about UUID usage.
  • perform some small corrections to text on Readme.md:

Meross Config
Meross Config is a config node to set the token, message id, timestamp (and UUID for Garage Door) being able to locally communicate with your devices.

Currently supported devices

  • Meross Smart Plug MSS210(EU)
  • Meross Garage Door MSG100

Text corrections are cosmetically and I hope there is okay to you. :-)

@STDevMan
Copy link
Contributor Author

STDevMan commented Feb 3, 2021

no problem. I will make the recommended changes.

I did some more testing and it looks like UUID isn't actually needed when making calls locally with an existing messageId and timestamp. I think it is required for meross cloud requests. It doesn't hurt to leave in UUID though but it can be any value. I tested with "uuid=hello" and it worked locally. I will add some code to randomly generate a size 16 hex string in case the code is reused for cloud stuff.

also I don't think the garage state call is working either, so I will get that working

The files now have the proper commands for checking garage door state.
@STDevMan STDevMan closed this Feb 3, 2021
@STDevMan STDevMan reopened this Feb 3, 2021
@STDevMan
Copy link
Contributor Author

STDevMan commented Feb 3, 2021

all files updated and tested in my main repo for Garage Door MSG100

@dehsgr dehsgr merged commit 4590a14 into dehsgr:main Feb 3, 2021
@dehsgr
Copy link
Owner

dehsgr commented Feb 3, 2021

Well done! I'll just published v1.1.0 to NPM. Thanks again! ;-)

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

2 participants