-
Notifications
You must be signed in to change notification settings - Fork 374
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
Implemented client side send operation #717
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the work. But I had some problems while trying it out
2e08ac6
to
02d94f7
Compare
The code looks good to me now. Thanks for your effort! Two improvements you could do for your commit:
|
6b8fe2d
to
77c25db
Compare
Commits squashed and commit message subject line changed. |
f66a04c
to
e18eaf5
Compare
Code formatted with |
Implemented send operation for client side, send operation was introduced with the LwM2M 1.1 standard. This PR is based on work from @sbertin-telular, specifically on send branch: https://github.com/sbertin-telular/wakaama/tree/send In this PR a bug that caused a memory leak has been fixed: sbertin-telular#1
2198e17
to
d5c218c
Compare
Code formatted with |
Sorry for the delay. I did a quick smoke test and it works. |
Implemented send operation for client side, send operation was introduced with the LwM2M 1.1 standard.
This PR is based on work from @sbertin-telular, specifically on send branch:
https://github.com/sbertin-telular/wakaama/tree/send
In this PR a bug that caused a memory leak has been fixed:
sbertin-telular#1