Skip to content

Fix mDNS network change detection - #1240

Merged
gskjold merged 1 commit into
UtilitechAS:mainfrom
schjonhaug:fix/mdns-network-change-detection
Aug 27, 2026
Merged

Fix mDNS network change detection#1240
gskjold merged 1 commit into
UtilitechAS:mainfrom
schjonhaug:fix/mdns-network-change-detection

Conversation

@schjonhaug

Copy link
Copy Markdown
Contributor

Summary

  • treat mDNS enablement changes as network changes so saving the setting triggers the required reboot
  • add a native AmsConfiguration regression suite backed by an in-memory EEPROM stub
  • run the new configuration suite in pull-request and main-branch CI

Testing

  • pio test -e native_config
  • verified red/green: the mDNS assertion fails without the production fix and passes with it
  • pio run -e esp8266
  • pio run -e esp32s2

@ArnieO

ArnieO commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Awaiting @gskjold assessment.

@gskjold
gskjold merged commit 3f2be2a into UtilitechAS:main Aug 27, 2026
2 of 8 checks passed
@schjonhaug
schjonhaug deleted the fix/mdns-network-change-detection branch August 27, 2026 07:21
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