search

UniversalDevicesInc / UD-Mobile-Android

Issue-only repository for tracking issues related to the UD Mobile Android App
0 stars 1 forks source link

Could not perform sync: Status Name - MUST BE A STRING #3

Closed DaveStLou closed 3 years ago

DaveStLou commented 3 years ago

Initial sync is failing with the following message: COULD NOT PERFORM SYNC Error: java.lang.illegalArgumentException: Nls Stats Names - Name - MUST BE A STRING

Only option is to cancel sync.

Screenshot_20201129-145038

JavierRefuerzo commented 3 years ago

Hi @DaveStLou, I added all Go2ISY testers as Alpha testers for UD Mobile and Google is serving some email list testers with the wrong version. Please verify app version number as I think this issue was resolved during internal testing. I removed all email testers just now but it has been a persistent issue. Thanks!

daveinglis commented 3 years ago

I have the same issue with version 0.27

JavierRefuerzo commented 3 years ago

Hi @daveinglis ,

I believe this is a node server missing status name causing the issue. Status will now be named "Node Serve Sync Issue" and an error will be presented to the user to notify the Node Server Dev.

Pushed version 0.28, please do let me know if the issue is resolved and which node server is causing the issue. Thanks

daveinglis commented 3 years ago

Just tried 0.28 and got another (similar) error: Screenshot_20201130-114137

JavierRefuerzo commented 3 years ago

Thanks, Pushed update 0.29 to catch missing command name, will now be raw value. As before please do let me know the node server so it can be reviewed.

daveinglis commented 3 years ago

0.29 sync succeeded. The nodeserver that it reported as having issues is Number 5 which looks to be the MQTT node server in my polyglot instance (https://github.com/exking/udi-mqtt-poly.git). Note, I don't have it configured with any MQTT endpoints.

Screenshot_20201130-132409

JavierRefuerzo commented 3 years ago

Hi @daveinglis ,

Thanks for your help resolving this issue. It appears these 3 names are listed as "Name" instead of "NAME" will run this by the others to see if we should accept both cases or request it be corrected.

DaveStLou commented 3 years ago

Hi @JavierRefuerzo

I'm running version 0.29 now and I'm no longer getting the original error.

Instead I'm getting the error below. FYI The referenced NodeServer is MQTT.

Screenshot_20201130-162514

JavierRefuerzo commented 3 years ago

Thanks @DaveStLou , This should not affect functionality aside from these three command names not having human readable values. It appears these 3 names are listed as "Name" instead of "NAME" in NLS, will run this by the others to see if we should accept both cases or request it be corrected.

JavierRefuerzo commented 3 years ago

@daveinglis , @DaveStLou Per Chris these values should be upper case as they may not be accepted in the ISY in all places. I have created a pull request in MQTT fixing this issue. https://github.com/exking/udi-mqtt-poly/pull/7#issue-530510101