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

Usability Improvements for esp32 platform #934

Merged
merged 4 commits into from
Apr 29, 2021
Merged

Usability Improvements for esp32 platform #934

merged 4 commits into from
Apr 29, 2021

Conversation

NorthernMan54
Copy link
Collaborator

I had been using these for a while with my devices, and wanted to share these usability improvements for others

1 - Ability to set SERIAL_BAUD via a compiler directive and platformio ini file, to allow using the higher serial speeds available with the ESP platform.

2 - In ESPWifiManualSetup add the ability to define a second SSID/Password via compiler directive wifi_ssid1/wifi_password1 and have the unit select the available one on startup. For both ESP8266 and ESP32 platforms.

3 - Add support for MDNS_SD for lookup of mqtt service address on ESP32 platform.

4 - Resolve issue with MDNS_SD not working on esp8266 platform. Arduino OTA was initializing MDNS and causing MDNS.begin to fail. As this feature may not be widely used I'm not sure if this issue was reported.

Northern Man added 4 commits April 27, 2021 16:59
1 - Expose SERIAL_BAUD as an optional compiler directive.
2 - Enabled mDNS based MQTT Host name lookup for ESP32 devices.
3 - Expand capability of ESPWifiManualSetup mode for multiple SSID’s.
Tested with ESP32
@1technophile 1technophile added this to the v0.9.7 milestone Apr 28, 2021
@1technophile
Copy link
Owner

Thanks for the PR!

@1technophile
Copy link
Owner

Did you test it in different configurations, with/without MDNS, with/without advanced network config?

@NorthernMan54
Copy link
Collaborator Author

Yes had tested both configurations on ESP32 and ESP8266

@1technophile 1technophile merged commit 2744510 into 1technophile:development Apr 29, 2021
@NorthernMan54 NorthernMan54 deleted the development-housekeeping branch April 29, 2021 01:52
@NorthernMan54
Copy link
Collaborator Author

Tks, now onto the Active receiver switching feature

My backlog

[ ] Active switching between signal receivers ( RF, PiLight and RTL_433 )
[ ] Home Assistant Discovery for RTL_433
[ ] Black/White listing for RTL_433
[ ] FSK Modulation support for rtl_433_ESP ( waiting for feedback on the feature )

1technophile pushed a commit that referenced this pull request May 11, 2021
* PR #934

* Active Receiver Switching

Switching of the active receiver module is available between the RF,
RF2, RTL_433 and Pilight Gateway modules, allowing for changing of
signal decoders without redploying the openMQTTGateway package.
Sending a JSON message to the command topic of the desired receiver
will change the active receiver module.

Other minor adjustments/improvements

1 - Improved startup serial logging that includes
- OpenMQTTGateway Version
- OpenMQTTGateway Modules

2 - Trace logging of full Gateway module command topic
 - Enabled in RF, RF2, PiLight and RTL_433

3 - Updated RTL_433 Documentation to only include enabled ASK/OOK
device decoders

4 - Trace logging of MQTT Callback topic

* Regression testing

* Adding valueAsASubject support to rtl_433_ESP
- model and id keys are appended to message topic

* Increased memory size for receiver switching in blocking build all

Co-authored-by: Northern Man <[email protected]>
@Legion2
Copy link
Contributor

Legion2 commented May 24, 2021

This broke the ESPWifiManualSetup feature in combination with the SECURE_CONNECTION feature, because WiFiMulti wifiMulti; is not defined when SECURE_CONNECTION is enabled but it is used in ESPWifiManualSetup.

Legion2 added a commit to Legion2/OpenMQTTGateway that referenced this pull request May 24, 2021
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