Skip to content

Conversation

@ccall48
Copy link
Contributor

@ccall48 ccall48 commented May 21, 2025

started adding in some dragino devices we use.

Copy link
Contributor

@brocaar brocaar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think in one of the codec scripts the decodeUplink function is missing. What is also still missing is the firmware(s) for each device. This is needed as this "glues" the device against one or multiple profiles and the codec.

See for example one of the RAK devices:

image

@ccall48
Copy link
Contributor Author

ccall48 commented Jun 2, 2025

I see, i didn't add in any firmware version so i would of missed that bit didn't realise it was mandatory.

give be a week or so and I'll try to get the firmware versions and tidy that side up.

I will check the payload decoders again some provided multiple initial functions for different lns's will confirm and fix if not.

@brocaar
Copy link
Contributor

brocaar commented Jun 2, 2025

Thanks @ccall48 👍

@brocaar
Copy link
Contributor

brocaar commented Jun 5, 2025

The tests are failing because:

  > Device: lht65n-e31f.toml
    > Firmware: 1.8
      > Codec: lht65n-e31f.js
Error: No such file or directory (os error 2)

@ccall48
Copy link
Contributor Author

ccall48 commented Jun 5, 2025

hmm one of my last commits should of removed that file and replaced it with lht65n.toml

@ccall48
Copy link
Contributor Author

ccall48 commented Jun 5, 2025

Sorry i forgot to commit the deletion my bad.. last pr update should fix this 🤞

@brocaar brocaar merged commit 0509f5e into chirpstack:master Jun 23, 2025
1 check passed
@brocaar
Copy link
Contributor

brocaar commented Jun 23, 2025

@ccall48 sorry for the delay, I just merged in your changes 🎉 Thanks for all your work!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants