arendst / Tasmota

Alternative firmware for ESP8266 and ESP32 based devices with easy configuration using webUI, OTA updates, automation using timers or rules, expandability and entirely local control over MQTT, HTTP, Serial or KNX. Full documentation at
https://tasmota.github.io/docs
GNU General Public License v3.0
22.24k stars 4.82k forks source link

DevGroupName: doesn't work with dashes in the name. Fix or document #9863

Closed hipitihop closed 4 years ago

hipitihop commented 4 years ago

PROBLEM DESCRIPTION

When using group name which include characters like dash - devices don't join the device group. Verified by issuing DevGroupStatus which would show "Members" as empty array [ ] Example: patio-leds does not work however pled works fine.

REQUESTED INFORMATION

Make sure your have performed every step and checked the applicable boxes before submitting your issue. Thank you!

- [ ] Provide the output of this command: `Status 0`:

STATUS 0 output here:

- [ ] Provide the output of the Console log output when you experience your issue; if applicable:
  _(Please use_ `weblog 4` _for more debug information)_

Console output here:



### TO REPRODUCE
Set `DevGroupName1 patio-led`

### EXPECTED BEHAVIOUR
Expect devices in this the said group to join and show as "Members" in `DevGroupStatus`

### SCREENSHOTS
_If applicable, add screenshots to help explain your problem._

### ADDITIONAL CONTEXT
_Add any other context about the problem here._

**(Please, remember to close the issue when the problem has been addressed)**
pcdiem commented 4 years ago

I don't see this behavior. Do you see the correct group name (patio-leds) for DevGroupName1 on all devices in the group when you enter the DevGroupName command?

18:53:53 MQT: stat/TestLight1/RESULT = {"DevGroupStatus":{"Index":0,"GroupName":"patio-leds","MessageSeq":4,"MemberCount":1,"Members":[{"IPAddress":"192.168.0.251","ResendCount":0,"LastRcvdSeq":2,"LastAckedSeq":4}]}}

ascillato2 commented 4 years ago

When you have time, please, let us know if this solves your issue. Thanks.

hipitihop commented 4 years ago

I have retested by assigning both devices to patio-leds and after they restart they indeed join the group and show as members. I'm puzzled with what I saw but indeed it now works as expected so my apologies for wasting peoples time.