Discovery / publish initial values

[iprak]

This PR address the issue of devices initially appearing unavailable. The MQTT publish for discovery has been adjusted to allow executing a command at the end, the command being publishChannels.

MQTT_PublishAll and MQTT_PublishChannels were reversed and have been addressed as well.

This should close #252 as well.

[openshwprojects]

Passing a pointer to a string that has to be kept in memory is not a good practice, but... I understand that it might be necessary here. Doing a strdup or using a buffer would not be worth the effort.

The only other approach I'd consider would be just creating an enum , something like PostPublishCommand with options None, PublishAll, PublishChannels and using it instead of string command.... maybe it would be better....

because otherwise people might be tempted to pass a local char buffer[64]; as an argument and be surprised that it crashes

[iprak]

Yes that is a good idea. I have revised my changes.

[iprak]

@openshwprojects The BL602 build is failing. This commit (716133a) introduced the function add_dimmer with no return value.

[openshwprojects]

@iprak I just realized that and fixed, thx

[github-actions]

🎉 This PR is included in version 1.14.38 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀