iotivity-lite
iotivity-lite copied to clipboard
add Push Notification support (CR2483)
Push Notification support for iotivity-lite (CR2483)
CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅
I have read the CLA Document and I hereby sign the CLA
Would it be possible to create an example application in apps directory? This is an extensive feature and I don't see any documentation or example, so I'm not sure people will know how to use it.
Also, I've added SonarCloud scan for this repo - it does some stylistic and linting checks, though I'm not sure the job will start for a fork. We will see when you rebase, I'd like the keep the amount of new issues reported by SC to zero.
I don't see coding problems in the pushed commits, but I haven't checked adherence to the specification... I guess there is nobody else to do it, eh?
Would it be possible to create an example application in apps directory? This is an extensive feature and I don't see any documentation or example, so I'm not sure people will know how to use it.
=>
Sure, I'll do it.
Where do you want me to add documentation?
Add to existing readme.md
, or create separate one ?
Also, I've added SonarCloud scan for this repo - it does some stylistic and linting checks, though I'm not sure the job will start for a fork. We will see when you rebase, I'd like the keep the amount of new issues reported by SC to zero.
I don't see coding problems in the pushed commits, but I haven't checked adherence to the specification... I guess there is nobody else to do it, eh?
=> I created CR3530 - Test case for Bug2483 Bug2658 and 2 CTT test case codes. The 2 CTT test case codes have already been merged to ctt-files repo. Comarch reviewed and merged them.
=> Sure, I'll do it. Where do you want me to add documentation? Add to existing
readme.md
, or create separate one ?
I don't want the main readme.md to be too long, so not there. Are you creating a demo app? If so I see that there is an .md file in the apps folder, where the demo application are described. If you want to create just documentation then I would create a new folder 'doc' and create a new .md file there which documents the feature.
=> Sure, I'll do it. Where do you want me to add documentation? Add to existing
readme.md
, or create separate one ?I don't want the main readme.md to be too long, so not there. Are you creating a demo app? If so I see that there is an .md file in the apps folder, where the demo application are described. If you want to create just documentation then I would create a new folder 'doc' and create a new .md file there which documents the feature.
It looks like that apps/Readme.md
is not a good place to describe additional feature such as push notification. So I think creating new doc
folder is better idea.
It looks like that
apps/Readme.md
is not a good place to describe additional feature such as push notification. So I think creating newdoc
folder is better idea.
Another alternative would be to put it in https://github.com/iotivity/iotivity.github.io . I think both options are good, pick the one you like.
It looks like that
apps/Readme.md
is not a good place to describe additional feature such as push notification. So I think creating newdoc
folder is better idea.Another alternative would be to put it in https://github.com/iotivity/iotivity.github.io . I think both options are good, pick the one you like.
Hi Daniel, I pushed new commit including examples & documentations. If you think that it looks ok, I'll rebase final commit and push it again.
~format
Closed since the other PR with these commits squashed was merged (github won't delete this PR, so you can still find it if you need to see what work was done).