Skip to content

Fixed SendDataOption in P031_SHT1X.ino - #5639

Merged
TD-er merged 1 commit into
letscontrolit:megafrom
BrainHunter:Fix_SHT1x_SendDataOption
Oct 7, 2026
Merged

TD-er merged 1 commit into
letscontrolit:megafrom
BrainHunter:Fix_SHT1x_SendDataOption

Conversation

@BrainHunter

Copy link
Copy Markdown

Fixed P031_SHT1X.ino to set SendDataOption to true since it was missing. This missing flag prevented the device configuration setting "Send to Controler" to be configured.
Now the device can be configured to send data as expected.

This allows the device to be configured to send data as expected.
@TD-er

TD-er commented Sep 20, 2026

Copy link
Copy Markdown
Member

Have you also checked other plugins to see if this was missing on those that actually generate data to be sent to a controller?

A while ago, Ton optimized this code to have proper defaults in the struct so only those plugins that needed something other than the default would need to have such a line in the PLUGIN_ADD call.
Apparently this was missing, but maybe it is missing in other plugins too?

@tonhuisman

Copy link
Copy Markdown
Contributor

A while ago, Ton optimized this code to have proper defaults in the struct

Well, it looks like I accidentally removed that option, quite some time ago, though I really can't recall why that would have been necessary, so it's probably correct that it's restored 🤔

@TD-er
TD-er merged commit a31a417 into letscontrolit:mega Oct 7, 2026
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