Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Update rcm-lrw10.js to pass correct return structure #786

Merged
merged 1 commit into from May 8, 2024

Conversation

ant2alex
Copy link
Contributor

@ant2alex ant2alex commented May 8, 2024

Summary

Fixes uplink decoder for rcm-lrw10. It did not return a correct structure so we only got "{}" as a result. This fixes it.

Changes

Changed formatting of file and return structure.

  • update rcm-lrw10.js

Checklist for Reviewers

  • Title and description should be descriptive (Not just a serial number for example).
  • profileIDs should not be vendorID and should be a unique value for every profile.
  • All devices should be listed in the vendor's index.yaml file.
  • Firmware versions can not be changed.
  • At least 1 image per device and should be transparent.

Notes for Reviewers

can be tested with this payload:
023041438707050300000013BB00000000E039CB

@Jaime-Trinidad Jaime-Trinidad merged commit 365e980 into TheThingsNetwork:master May 8, 2024
2 checks passed
@ant2alex ant2alex deleted the patch-1 branch May 8, 2024 12:51
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.

None yet

2 participants