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

SetOption158 publish/suppress ModbusReceived MQTT messages #20678

Merged
merged 1 commit into from
Feb 7, 2024
Merged

SetOption158 publish/suppress ModbusReceived MQTT messages #20678

merged 1 commit into from
Feb 7, 2024

Conversation

FulvioSpelta
Copy link
Contributor

Description:

Related issue (if applicable): fixes #

Checklist:

  • The pull request is done against the latest development branch
  • Only relevant files were touched
  • Only one feature/fix was added per PR and the code change compiles without warnings
  • The code change is tested and works with Tasmota core ESP8266 V.2.7.6
  • The code change is tested and works with Tasmota core ESP32 V.2.0.14
  • I accept the CLA.

NOTE: The code change must pass CI tests. Your PR cannot be merged unless tests pass

@FulvioSpelta
Copy link
Contributor Author

Implement the request from Q/A: #20501

@arendst arendst merged commit ab8676d into arendst:development Feb 7, 2024
64 checks passed
hawa-lc4 pushed a commit to hawa-lc4/Tasmota-dev that referenced this pull request Feb 12, 2024
@UBWH
Copy link
Contributor

UBWH commented Feb 18, 2024

Installed 13.4.0
image
image

Tried SETOPTION158 ON and OFF
Made no difference?

17:17:45.614 CMD: setoption158 on
17:17:45.620 MQT: stat/solar_06C6B8/RESULT = {"SetOption158":"ON"}
17:17:38.777 MQT: tele/solar_06C6B8/RESULT = {"ModbusReceived":{"DeviceAddress":1,"FunctionCode":4,"StartAddress":12800,"Length":11,"Count":3,"Values":[0,7,1]}}
....
17:17:45.614 CMD: setoption158 off
17:17:45.620 MQT: stat/solar_06C6B8/RESULT = {"SetOption158":"OFF"}
17:17:45.786 MQT: tele/solar_06C6B8/RESULT = {"ModbusReceived":{"DeviceAddress":1,"FunctionCode":2,"StartAddress":8192,"Length":6,"Count":1,"Values":[41216]}}
17:17:46.814 MQT: tele/solar_06C6B8/RESULT = {"ModbusReceived":```

@FulvioSpelta
Copy link
Contributor Author

There is a fix in 13.4.0.1 #20733

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