-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
samples: Matter: light_bulb: Add support for AWS IoT #12876
Conversation
fcf6fb6
to
6c5b09d
Compare
You can find the documentation preview for this PR at this link. It will be updated about 10 minutes after the documentation build succeeds. Note: This comment is automatically posted by the Documentation Publishing GitHub Action. |
6ae6598
to
470da91
Compare
dbf0efc
to
7151116
Compare
7151116
to
1c02b91
Compare
The following west manifest projects have been modified in this Pull Request:
Note: This message is automatically posted and updated by the Manifest GitHub Action. |
930050e
to
c561f9b
Compare
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 your contribution. I left some initial comments from the integration perspective. I think that we also miss:
- README update with information how this build could be enabled and tested
- Entry in samples.yml that would at least verify the building process passes.
c561f9b
to
9eaa2bd
Compare
I am working on the figures and reviewing the documents. @wiba-nordic - Maybe you could take a look once I complete my part of the review? I will keep you posted. |
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.
Sample yaml looks good.
Test specificationCI/Jenkins/NRF
CI/Jenkins/integration
Detailed information of selected test modules Note: This message is automatically posted and updated by the CI |
9c50f11
to
d3bc74a
Compare
acba916
to
d84bd44
Compare
d84bd44
to
3521827
Compare
a5c650a
to
d2c0d45
Compare
d2c0d45
to
f013f7b
Compare
f013f7b
to
5e0e024
Compare
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.
Little things I noticed looking over this again.
cb15616
to
3413788
Compare
5216e54
to
c4272fa
Compare
Add types for subscribed topics and include the type in the received payload. Makes it easier to perform different actions based on topic type in the application. Signed-off-by: Simen S. Røstad <[email protected]>
Add support for AWS IoT Signed-off-by: Simen S. Røstad <[email protected]>
c4272fa
to
d9647ba
Compare
Add support for AWS IoT