TheThingsNetwork / lorawan-devices

Device Repository for LoRaWAN devices
Apache License 2.0
193 stars 373 forks source link

Error while validating #715

Closed reghund1986 closed 8 months ago

reghund1986 commented 10 months ago

HI ,

iam facing some issue while validating the code . "Error: Command failed: bin/validate-image ./vendor/adeunis/tic.png 'bin' is not recognized as an internal or external command, operable program or batch file. " Are there any packages to be installed? Could you please help me to resolve the issue Regards reghu TTN Bug

Summary

...

What do you see now?

...

What do you want to see instead?

...

How do you propose to implement this?

...

Can you do this yourself and submit a Pull Request?

...

KrishnaIyer commented 10 months ago

@LDannijs, could you please take a look?

LDannijs commented 10 months ago

Could you make sure you're running Node.js version 16.x and install dependencies after with make deps

reghund1986 commented 10 months ago

HI ,

Now I can validate the project without error .

Is there any coding standard to be follow for .js . Am getting below error message.

TTN Bug_1

Hi ,

I tested the javascript manually , its giving proper output in json format .

{"data":{"id":109,"type":4,"seq_counter":19,"fw_version":1,"values":{"temperature":{"unit":"oc","value":22.9},"humidity":{"unit":"%","value":28.8}},"alarm_status":{"temperature":{"high":false,"low":false},"humidity":{"high":false,"low":true}},"states":{"battery":"100%","msg_type":"normal"}}}

Regards Reghu

LDannijs commented 10 months ago

We have a rundown on making javascript formatters in our documentation: https://www.thethingsindustries.com/docs/integrations/payload-formatters/javascript/

If you have more questions feel free to ask.

LDannijs commented 8 months ago

Hello @reghund1986, is this issue fixed? If so we can close it.

reghund1986 commented 8 months ago

Hello @reghund1986, is this issue fixed? If so we can close it.

Yes , its validating successfully .

LDannijs commented 8 months ago

Great to hear! Closing this issue.