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

BLE passive MQTT GW + Shelly2PM Domoticz cover fix #59

Merged
merged 9 commits into from
Dec 12, 2023

Conversation

enesbcs
Copy link
Contributor

@enesbcs enesbcs commented Aug 19, 2023

No description provided.

@enesbcs enesbcs changed the title Shelly2PM Domoticz cover fix BLE passive MQTT GW + Shelly2PM Domoticz cover fix Nov 20, 2023
@Mircho
Copy link
Contributor

Mircho commented Dec 12, 2023

Is there a difference between the two files - v1 and v2? If the second one is an improvement, then can we drop the first?

Additionally - please add description of the examples provided in the manifest file - json

@Mircho
Copy link
Contributor

Mircho commented Dec 12, 2023

And first and foremost - thank you for your contribution!

@enesbcs
Copy link
Contributor Author

enesbcs commented Dec 12, 2023

Is there a difference between the two files - v1 and v2? If the second one is an improvement, then can we drop the first?

Yes, v2 uses try-catch which requires Shelly firmware >=1.0.0
v1 does not use try-catch so it will work fine with 0.x firmwares also, can be dropped if 0.x firmwares is not supported anymore.

Additionally - please add description of the examples provided in the manifest file - json

Sure, i fill the json, thanks for pointing to it!

@Mircho Mircho merged commit f97b1ca into ALLTERCO:main Dec 12, 2023
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.

3 participants