Skip to content
This repository has been archived by the owner on Sep 10, 2024. It is now read-only.

elaboarate on Google Cloud Platform MQTT sample code #119

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

netskink
Copy link

What does this implement/fix? Explain your changes.

This is a fix for this issue
#118

Does this close any currently open issues?

Yes. It closes the above issue. The existing documentation needs elaboration.

Any relevant logs, error output, etc?

None

Any other comments?

This shows that the genkey.sh script creates a key for the device and not the topic. It also shows screenshots of the google cloud platform setup pages and gives detail on how the topic name 'events' seems to be required. Without this name, only pushes work and the device configuration pull in the final part of the code fails.

Where has this been tested?

  • lopy4 and exapansion board 3.1
  • (sysname='LoPy4', nodename='LoPy4', release='1.20.2.rc6', version='v1.20.1.r2-114-ge264b4c24-dirty on 2020-04-21', machine='LoPy4 with ESP32', lorawan='1.0.2', sigfox='1.0.1')

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants