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

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

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