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

LW examples #1034

Merged
merged 16 commits into from
Mar 25, 2024
Merged

LW examples #1034

merged 16 commits into from
Mar 25, 2024

Conversation

HeadBoffin
Copy link
Collaborator

Just the Starter, ABP and Reference.

There is a winkle with ABP nomclementure & the order of parameters between the TTN console and beginABP. The config.h matches the Overview on console but the parameter names don't currently match under the hood. This will only be spotted by someone looking at the code before @StevenCellist corrects it.

There is also the possibility that we can drop the force=true on beginOTAA if there is no session restored from persistence.

Despite the above, this moves things forward and more importantly, some actual proper device testing can be done on other platforms (AVR, SAMD, STM32) and, for the other repro, persistence as well as working on the wiki.

Feel free to enact any changes to get these over the wire please.

@StevenCellist
Copy link
Collaborator

StevenCellist commented Mar 25, 2024

@HeadBoffin @jgromes looks all good to me. I prepared a README for the examples/LoRaWAN folder that I will add as soon as the PR is merged. Done now.

Copy link
Owner

@jgromes jgromes left a comment

Choose a reason for hiding this comment

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

Everything looks good to me. If there are no further changes then I think we can merge this and finally get 6.5.0 out!

@StevenCellist StevenCellist merged commit 9c6eb20 into jgromes:master Mar 25, 2024
30 checks passed
@StevenCellist
Copy link
Collaborator

That wasn't a nice squashed merge, oops, sorry, but it's in :)

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.

4 participants