Skip to content

Update README and MQTT security settings to support updated Sling protocol#909

Merged
YaleChen299 merged 3 commits intosource-academy:masterfrom
RichDom2185:richard/update-readme-and-mqtt-security-settings
Sep 25, 2022
Merged

Update README and MQTT security settings to support updated Sling protocol#909
YaleChen299 merged 3 commits intosource-academy:masterfrom
RichDom2185:richard/update-readme-and-mqtt-security-settings

Conversation

@RichDom2185
Copy link
Member

Related PRs:

Although the abovementioned PRs are related, this PR is safe to merge independently.

Changes:

  • Remove hard breaks in + reformat README (the former was causing rendering issues as they were interpreted as separate lines)
  • Improves the developer experience by providing a guide on how to set up a local MQTT server to test remote execution locally
  • Update terraform whitelists for MQTT topics to support the updated Sling version (containing a new message topic, monitor)

@coveralls
Copy link

Coverage Status

Coverage remained the same at 95.997% when pulling 3482eda on RichDom2185:richard/update-readme-and-mqtt-security-settings into b25ebce on source-academy:master.

@RichDom2185 RichDom2185 marked this pull request as ready for review September 21, 2022 19:35
@RichDom2185 RichDom2185 self-assigned this Sep 22, 2022
@YaleChen299 YaleChen299 merged commit cb91f7e into source-academy:master Sep 25, 2022
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.

3 participants