Support data.world as a datasource

[rflprr]

data.world would be a great general purpose, low entry barrier datasource for plot.ly.
It should be a very straightforward integration based on the implementation of existing data.world connectors for Tableau (https://github.com/datadotworld/tableau-connector) and Google Data Studio (https://github.com/datadotworld/data-studio-connector).

[bpostlethwaite]

That is a great idea, thanks for the feedback!

[n-riesco]

@rflprr I'm labelling this issue as community. I also think this is a good first contribution. If you feel like contributing, I can point you to the files that need changing and help through the process.

[rflprr]

Hi, @n-riesco. We would like to contribute and could start as early as this week. What's the best way for us to get started?

[n-riesco]

@rflprr

First of all, you need to set up your dev machine:

• install Node.js v6
• install yarn v1
• clone Falcon's repository, build and run:

  $ git clone https://github.com/plotly/falcon-sql-client.git
  $ cd falcon-sql-client
  $ yarn install
  $ yarn run rebuild:modules:electron
  $ yarn run build
  $ yarn run start-hot
  

Once the dev machine is setup. Here's a list of files you need to look at to add a new connector:

• backend/persistent/datastores/Datastores.js
• app/constants/constants.js
• app/images
• app/actions/sessions.js
• app/components/Settings/Tabs/Tab.react.js
• app/components/Settings/Settings.react.js

As an example of what needs doing, you could also have a look at the following commits from the PR implementing the Impala connector:

• UI changes: 2d1477f
• Backend implementation: 67ae51d
• Add a sample connection: 9e85a2b
• Add tests: https://github.com/plotly/falcon-sql-client/blob/master/test/backend/datastores.ibmdb.spec.js

Hopefully, I didn't miss anything ;), but if I did or you need more help, don't hesitate to contact me.

[kndungu]

@n-riesco

I've just made a PR for the initial implementation.

Pending:

I've as yet been unable to make the database name appear on the side panel's tree view, might you have some thoughts on this?

This is now addressed in bcb7315, thoughts still welcome.

Token:
Connection to a dataset requires the user to provide a valid data.world API token. Due to various reasons we'd prefer not to provide a publicly accessible token. This is the reason as to why tests are not part of the PR. Your thoughts on this would also be appreciated

cc @rflprr

[n-riesco]

Closed via #332