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

Fix up documentation examples #119

Merged
merged 2 commits into from
Nov 8, 2018
Merged

Fix up documentation examples #119

merged 2 commits into from
Nov 8, 2018

Conversation

xginn8
Copy link
Contributor

@xginn8 xginn8 commented Nov 7, 2018

Change-type: patch
Signed-off-by: Matthew McGinn mamcgi@gmail.com

Front logo Front conversations

DOCUMENTATION.md Outdated
@@ -245,7 +245,7 @@ Tethering=false

[Bluetooth]
Enable=true
Tethering=false'}, u'pubnubPublishKey': u'pub-c-6cbce8db-bfd1-4fdf-a8c8-53671ae2b226', u'apiEndpoint': u'https://api.balena.io', u'connectivity': u'connman', u'deviceType': u'raspberrypi3', u'mixpanelToken': u'12345678912345678912345678912345', u'deltaEndpoint': u'https://delta.balena.io', u'appUpdatePollInterval': 60000, u'applicationId': 106640, u'registryEndpoint': u'registry.balena.io'}
Tethering=false'}, u'pubnubPublishKey': u'pub-c-6cbce8db-bfd1-4fdf-a8c8-53671ae2b226', u'apiEndpoint': u'https://api.balena.io', u'connectivity': u'connman', u'deviceType': u'raspberrypi3', u'mixpanelToken': u'99eec53325d4f45dd0633abd719e3ff1', u'deltaEndpoint': u'https://delta.balena.io', u'appUpdatePollInterval': 60000, u'applicationId': 106640, u'registryEndpoint': u'registry.balena.io'}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you please change the mixpanelToken to a random string? We don't want to put real token here, I guess someone fixed the docs but haven't change the example in the source, it's here https://github.com/balena-io/resin-sdk-python/blob/master/balena/models/application.py#L274

@nghiant2710
Copy link
Contributor

@xginn8, can you merge this PR with #120, I think they can be merged.

I only have one small comment, otherwise LGTM 👍

Change-type: patch
Signed-off-by: Matthew McGinn <mamcgi@gmail.com>
Change-type: patch
Signed-off-by: Matthew McGinn <mamcgi@gmail.com>
@xginn8 xginn8 merged commit 31e8211 into master Nov 8, 2018
@xginn8 xginn8 deleted the docs_fixup branch November 8, 2018 20:34
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.

2 participants