espressif / esp-insights

ESP Insights: A remote diagnostics/observability framework for connected devices
Apache License 2.0
101 stars 27 forks source link

"Please provide a valid package name" #17

Closed ivancmz closed 1 year ago

ivancmz commented 1 year ago

When trying to add the Firmware Package on the dashboard I get the "Please provide a valid package name" error.

Captura de pantalla 2022-11-29 213231

The Project name is: R84.24 The version is: 2_0_0_221118 The zip file generated is: R84.24-v2_0_0_221118.zip The package name I'm using is: R84.24-v2_0_0_221118, also tried R84.24-2_0_0_221118 and other combinations.

Captura de pantalla 2022-11-29 213313

ivancmz commented 1 year ago

Changed version to a more "traditional" scheme (i.e., R84.24-2.0), same issue.

Changed project name to R84_24 to avoid dots (i.e., R84_24-2.0), same issue (R84_24-2.0).

Changed project name to Light to match exactly the dialogs example values (Light-2.0) and it still fails.

image

ivancmz commented 1 year ago

It finally worked after removing the version part, that is package name "Light" without the "-2.0"

shahpiyushv commented 1 year ago

@ivancmz , good to know you could get it working but we do understand that this is very inconvenient. The package name can actually be any friendly name that the user wants, independent of actual project name and version. We will fix this bug asap.

ivancmz commented 1 year ago

@ivancmz , good to know you could get it working but we do understand that this is very inconvenient. The package name can actually be any friendly name that the user wants, independent of actual project name and version. We will fix this bug asap.

Thanks @shahpiyushv. What I've seen is that it only allows letters in the name, no numbers or special characters like underscore. I see now that I can use any name, but since the placeholder example was "Light-2.0" I assumed it had to match the project name and version.

adwait-esp commented 1 year ago

This issue was caused by some regression, it has been fixed and pushed to production. @ivancmz can you please check and close if the issue is fixed?