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

GraffSDK a Requires? Avoiding future facepalms #33

Closed
dehann opened this issue Mar 31, 2019 · 5 comments
Closed

GraffSDK a Requires? Avoiding future facepalms #33

dehann opened this issue Mar 31, 2019 · 5 comments

Comments

@dehann
Copy link
Member

dehann commented Mar 31, 2019

There is strong cause for users knowing precisely when 'the software' starts talking back to some server over the internet. This should only occur under the user's express desire to do so, for example along with a console message stating as much:

using GraffSDK # load first
using Arena    # cloud features are now enabled

# start cloud communication
cfg = GraffSDK.loadConfig(user...)
# printout to console confirming communication has started

...

Otherwise the entire code base might be perceived as 'not respecting privacy', precisely the opposite of what is intended!

cc @GearsAD

@GearsAD
Copy link
Contributor

GearsAD commented Mar 31, 2019

Not sure I see how Graff would be perceived as 'bad privacy behavior', but I fully get the optional requires.

@GearsAD
Copy link
Contributor

GearsAD commented Mar 31, 2019

You need a login and explicit request to get your data, it’s very different to Dash story. Graff is not pushing any data upstream unless you explicitly ask, also Arena is just pulling down for rendering so it’s a moot point. Like that it’s an extension, don’t quite understand the motivation though.

@dehann
Copy link
Member Author

dehann commented Mar 31, 2019

Right, so the answer then is we should just say that explicitly on the cover page, before someone else starts spreading misinformation. I'm scared of misinformation spreading.

@GearsAD
Copy link
Contributor

GearsAD commented Mar 31, 2019

To be explicit here: Graff in Arena just pulls down data

dehann added a commit that referenced this issue Mar 31, 2019
fixes public notification relating to #33
@dehann dehann mentioned this issue Mar 31, 2019
@dehann dehann changed the title Make GraffSDK a Requires based import, not a default GraffSDK a Requires? Avoiding future facepalms Mar 31, 2019
@dehann dehann closed this as completed Mar 31, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants