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.

Sign up for GitHub

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

Jump to bottom

IRMQTTServer: Add option to clear retained settings. #917

Merged
merged 2 commits into from
Sep 15, 2019
Merged

IRMQTTServer: Add option to clear retained settings. #917

merged 2 commits into from
Sep 15, 2019

Conversation

crankyoldgit
Copy link
Owner

  • Add option to clear retained settings. (Under the "admin" page)
  • Add HA discovery and LWT to things to be cleared as well.

Fixes #916

@crankyoldgit
IRMQTTServer: Add option to clear retained settings.
d189195 
Fixes #916

Add HA discovery and LWT to things to be cleared.
@crankyoldgit crankyoldgit self-assigned this Sep 15, 2019
@crankyoldgit
Create a flag to control clearing retained MQTT data.
6147935 
* Add `MQTT_CLEAR_ENABLE` define to control if this gets compiled in or 
not.
  It's disabled by default as most people shouldn't need this.
* Clean up the text for this option a little.

Note: If enabled, it costs about 1.1k of flash space.

Ref #916
@crankyoldgit crankyoldgit merged commit b8cd584 into master Sep 15, 2019
@crankyoldgit
Copy link
Owner Author

@sillyfrog Thanks for the prompt code review. Appreciated.

@crankyoldgit crankyoldgit deleted the clear_mqtt branch September 16, 2019 14:10
crankyoldgit added a commit that referenced this pull request Sep 23, 2019
@crankyoldgit
v2.6.6 release
9ca04ad 
_v2.6.6 (20190923)_

**[Bug Fixes]**
- Ensure `begin()` is called for every supported common a/c. (#905, #899)
- IRMQTTServer: Fix JSON state parsing. (#896)
- IRMQTTServer: Fix compilation error when `MQTT_CLIMATE_JSON` is `true`. (#893)

**[Features]**
- Mitsubishi136: Full A/C support. (#898, #890)
- Fujitsu: Add support for ARRY4 remote. (#895)
- Web-AC-control: Add new WebUI example sketch. (#880, #886)
- Improve Common A/C API (#913)
- IRMQTTServer: Support for multiple climates. (#903)
- IRMQTTServer: Add TX channel support for HTTP interface. (#929)
- IRMQTTServer: Add option to clear retained settings. (#917)
- auto_analyse_raw_data.py: Add decode code generation. (#909)
- auto_analyse_raw_data.py: General improvements (#906)

**[Misc]**
- IRMQTTServer: Use latest API for common A/C. (#928)
- IRMQTTServer: Add flag & documentation for Home Assistant mode. (#919)
- IRMQTTServer: Move from ArduinoJson v5 to v6. (#878)
- IRMQTTServer: Use retain for discovery message. (#881)
- Goodweather: Adjust timings & minor fixes. (#924)
- PanasonicAc: Add better SwingV support for common a/c framework. (#923)
- Daikin2: Corrections for common A/C interface. (#910)
- MitsubishiAC: Improve decoding. (#914)
- Fujitsu: Disable horiz swing for ARRY4. (#907)
- SamsungAc: Only send power on/off code if it's needed. (#884)
- Teco: Add timer support. (#883)
- More consistent A/C `::toString()` output. (#920)
@crankyoldgit crankyoldgit mentioned this pull request Sep 23, 2019
Merged
crankyoldgit added a commit that referenced this pull request Sep 23, 2019
@crankyoldgit
v2.6.6 release (#930)
b95793c 
_v2.6.6 (20190923)_

**[Bug Fixes]**
- Ensure `begin()` is called for every supported common a/c. (#905, #899)
- IRMQTTServer: Fix JSON state parsing. (#896)
- IRMQTTServer: Fix compilation error when `MQTT_CLIMATE_JSON` is `true`. (#893)

**[Features]**
- Mitsubishi136: Full A/C support. (#898, #890)
- Fujitsu: Add support for ARRY4 remote. (#895)
- Web-AC-control: Add new WebUI example sketch. (#880, #886)
- Improve Common A/C API (#913)
- IRMQTTServer: Support for multiple climates. (#903)
- IRMQTTServer: Add TX channel support for HTTP interface. (#929)
- IRMQTTServer: Add option to clear retained settings. (#917)
- auto_analyse_raw_data.py: Add decode code generation. (#909)
- auto_analyse_raw_data.py: General improvements (#906)

**[Misc]**
- IRMQTTServer: Use latest API for common A/C. (#928)
- IRMQTTServer: Add flag & documentation for Home Assistant mode. (#919)
- IRMQTTServer: Move from ArduinoJson v5 to v6. (#878)
- IRMQTTServer: Use retain for discovery message. (#881)
- Goodweather: Adjust timings & minor fixes. (#924)
- PanasonicAc: Add better SwingV support for common a/c framework. (#923)
- Daikin2: Corrections for common A/C interface. (#910)
- MitsubishiAC: Improve decoding. (#914)
- Fujitsu: Disable horiz swing for ARRY4. (#907)
- SamsungAc: Only send power on/off code if it's needed. (#884)
- Teco: Add timer support. (#883)
- More consistent A/C `::toString()` output. (#920)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sillyfrog sillyfrog sillyfrog approved these changes

@roidayan roidayan Awaiting requested review from roidayan

Assignees

@crankyoldgit crankyoldgit

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add clear all retained messages and reboot to IRMQTTServer
2 participants
@crankyoldgit @sillyfrog