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

[OTA] Enable 2 clicks OTA update / one simple MQTT command #1485

Merged
merged 2 commits into from
Feb 27, 2023

Conversation

1technophile
Copy link
Owner

Description:

This feature will scan at the start if an update is available, if yes an MQTT command can be triggered to launch the update
For the controllers using Home Assistant convention, the option to update will be presented, and the update will be done through a 2 clicks sequence
image

Checklist:

  • The pull request is done against the latest development branch
  • Only one feature/fix was added per PR and the code change compiles without warnings
  • I accept the DCO.

This feature will enable to update the gateway with one button or a simple MQTT command
@1technophile 1technophile force-pushed the auto-ota branch 3 times, most recently from 88f35ef to 0b257f1 Compare February 26, 2023 15:24
And integrate the latest_version file creation into the CI

Enable to specify a particular version
and switch between dev and production builds
@1technophile 1technophile merged commit 454a062 into development Feb 27, 2023
@1technophile 1technophile deleted the auto-ota branch February 27, 2023 18:54
jsondata["installed_version"] = OMG_VERSION;
jsondata["entity_picture"] = ENTITY_PICTURE;
latestVersion = jsondata["latest_version"].as<String>();
pub(subjectSYStoMQTT, jsondata);
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this be a separate topic ? As the message is lost/over written with further updates.

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is enough to be caught by HA, but for the sake of readiness I can understand the argument for a separate topic

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.

2 participants