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

samples: Matter: light_bulb: Add support for AWS IoT #12876

Merged
merged 2 commits into from
Dec 28, 2023

Conversation

simensrostad
Copy link
Contributor

@simensrostad simensrostad commented Oct 26, 2023

Add support for AWS IoT

@github-actions github-actions bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Oct 26, 2023
@simensrostad simensrostad force-pushed the matter-aws-iot-brige branch 2 times, most recently from fcf6fb6 to 6c5b09d Compare October 30, 2023 18:57
@NordicBuilder
Copy link
Contributor

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.

@simensrostad simensrostad force-pushed the matter-aws-iot-brige branch 12 times, most recently from 6ae6598 to 470da91 Compare November 6, 2023 17:25
@simensrostad simensrostad force-pushed the matter-aws-iot-brige branch 4 times, most recently from dbf0efc to 7151116 Compare November 8, 2023 17:53
@NordicBuilder
Copy link
Contributor

The following west manifest projects have been modified in this Pull Request:

Name Old Revision New Revision Diff
zephyr nrfconnect/sdk-zephyr@533baa1 nrfconnect/sdk-zephyr#1391 nrfconnect/sdk-zephyr#1391/files

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@simensrostad simensrostad force-pushed the matter-aws-iot-brige branch 2 times, most recently from 930050e to c561f9b Compare November 15, 2023 09:32
Copy link
Contributor

@kkasperczyk-no kkasperczyk-no left a 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.

@divipillai
Copy link
Contributor

divipillai commented Dec 13, 2023

@doublemis1 could you take a look at samples.yml and @wiba-nordic could you review the readme change?

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.

Copy link
Contributor

@doublemis1 doublemis1 left a 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.

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Dec 15, 2023

Test specification

CI/Jenkins/NRF

  • Integration Platforms

CI/Jenkins/integration

Test Module File based changes Manually selected West overwrite
test-ci-nrfconnect-boot-fw-update X
test-fw-nrfconnect-chip X
test-fw-nrfconnect-nrf-iot_thingy91 X

Detailed information of selected test modules

Note: This message is automatically posted and updated by the CI

@divipillai divipillai force-pushed the matter-aws-iot-brige branch 2 times, most recently from acba916 to d84bd44 Compare December 15, 2023 13:15
@simensrostad simensrostad force-pushed the matter-aws-iot-brige branch 2 times, most recently from a5c650a to d2c0d45 Compare December 15, 2023 14:48
Copy link
Contributor

@wiba-nordic wiba-nordic left a 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.

@simensrostad simensrostad force-pushed the matter-aws-iot-brige branch 2 times, most recently from 5216e54 to c4272fa Compare December 21, 2023 10:06
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]>
@cvinayak cvinayak merged commit d176649 into nrfconnect:main Dec 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc-required PR must not be merged without tech writer approval.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants