I've found that:
char *protocolMyMessage2Serial(const MyMessage &message)
{
(void)snprintf_P(_fmtBuffer, (uint8_t)MY_GATEWAY_MAX_SEND_LENGTH,
PSTR("%" PRIu8 ";%" PRIu8 ";%" PRIu8 ";%" PRIu8 ";%" PRIu8 ";%s\n"), message.getSender(),
message.getSensor(), message.getCommand(), message.isEcho(), message.getType(),
message.getString(_convBuffer));
return _fmtBuffer;
}
We put in the print "message.getEcho()", but this isn't echoed message.
I think we should put "message.getRequestEcho()" here.
@TheoL there are two pull requests adding support for newer versions.
One is guaranteed to break again.
One should not break in the future, but will instead break all existing sketches.
I’m sure a third option would be welcome if you write the code and submit.
I have mixed Nanos, Pro Minis, ESP8266 and Unos and no problem. I do however have mixed NRF24L01 and NRF24L01+ due to a bad AliExpress saler that stated he sold NRF24L01+ which in fact were not the + version. So I have set my baudrate on all nodes to 1mb.
In order to be helpful we need the log on the Uno.
