[nteract/web] Add notebook component and wrap up for beta and GSoC.

[ramantehlan]

This PR is to finish adding all the tasks required to launch it as beta and wrap up the GSoC period.

UI/UX Integration

The list of components that needs to be created and added is given below, more might be added later:

• Layout
• BinderMenu
• Input
• Menu
• Button
• Avatar
• Dialog
• Landing Page
• Auth page
• Editor Using @nteract/editor | Later we can use @nteract/monaco-editor
• Console
• FileExplorer
• Notification Using new component | Later we can use nteract legacy lib
• Notebook rendering.

Github Integration

This PR will also allow OAuth authentication with Github, and fetch user public info.

• Complete authentication
• Fetch user's public info
• Fetch public files and folders
• List and create a fork for repo
• Save back changes to the repo or the fork

MyBinder Integration

• Launch MyBinder instance on start.
• Communicate with JupyterServer. Fetch info like sessions etc.
• Execute the command on MyBinder instance
• Fetch content from Github and not jupyter server.
• Save back changes done to the notebook.

[lgtm-com]

This pull request fixes 2 alerts when merging e4920e5 into f109c9c - view on LGTM.com

fixed alerts:

• 2 for Unused variable, import, function or class

[lgtm-com]

This pull request fixes 2 alerts when merging 9b502d8 into f109c9c - view on LGTM.com

fixed alerts:

• 2 for Unused variable, import, function or class

[lgtm-com]

This pull request introduces 6 alerts and fixes 2 when merging 2bfece1 into f109c9c - view on LGTM.com

new alerts:

• 5 for Unused variable, import, function or class
• 1 for Use of returnless function

fixed alerts:

• 2 for Unused variable, import, function or class

[lgtm-com]

This pull request introduces 6 alerts and fixes 3 when merging 30de767 into 595b72b - view on LGTM.com

new alerts:

• 6 for Unused variable, import, function or class

fixed alerts:

• 3 for Unused variable, import, function or class

[lgtm-com]

This pull request introduces 6 alerts and fixes 3 when merging e3df7d4 into 595b72b - view on LGTM.com

new alerts:

• 6 for Unused variable, import, function or class

fixed alerts:

• 3 for Unused variable, import, function or class

[lgtm-com]

This pull request introduces 6 alerts and fixes 3 when merging 1ed4ca2 into 595b72b - view on LGTM.com

new alerts:

• 6 for Unused variable, import, function or class

fixed alerts:

• 3 for Unused variable, import, function or class

[lgtm-com]

This pull request introduces 5 alerts and fixes 5 when merging dc7d2cf into 595b72b - view on LGTM.com

new alerts:

• 4 for Unused variable, import, function or class
• 1 for Use of returnless function

fixed alerts:

• 5 for Unused variable, import, function or class

[lgtm-com]

This pull request introduces 5 alerts and fixes 5 when merging ac96384 into 595b72b - view on LGTM.com

new alerts:

• 4 for Unused variable, import, function or class
• 1 for Use of returnless function

fixed alerts:

• 5 for Unused variable, import, function or class

[lgtm-com]

This pull request introduces 5 alerts and fixes 5 when merging 29ffa0b into 595b72b - view on LGTM.com

new alerts:

• 4 for Unused variable, import, function or class
• 1 for Use of returnless function

fixed alerts:

• 5 for Unused variable, import, function or class

[lgtm-com]

This pull request introduces 5 alerts and fixes 5 when merging 499a216 into 595b72b - view on LGTM.com

new alerts:

• 4 for Unused variable, import, function or class
• 1 for Use of returnless function

fixed alerts:

• 5 for Unused variable, import, function or class

[lgtm-com]

This pull request introduces 6 alerts and fixes 5 when merging 637a709 into b87e929 - view on LGTM.com

new alerts:

• 4 for Unused variable, import, function or class
• 1 for Useless assignment to local variable
• 1 for Use of returnless function

fixed alerts:

• 5 for Unused variable, import, function or class

[lgtm-com]

This pull request introduces 10 alerts and fixes 5 when merging 15fa6e4 into b87e929 - view on LGTM.com

new alerts:

• 8 for Syntax error
• 1 for Useless assignment to local variable
• 1 for Use of returnless function

fixed alerts:

• 5 for Unused variable, import, function or class

[lgtm-com]

This pull request introduces 6 alerts and fixes 5 when merging f61edcf into b87e929 - view on LGTM.com

new alerts:

• 4 for Unused variable, import, function or class
• 1 for Useless assignment to local variable
• 1 for Use of returnless function

fixed alerts:

• 5 for Unused variable, import, function or class

[lgtm-com]

This pull request introduces 3 alerts and fixes 5 when merging 687a574 into b87e929 - view on LGTM.com

new alerts:

• 2 for Unused variable, import, function or class
• 1 for Useless assignment to local variable

fixed alerts:

• 5 for Unused variable, import, function or class

[lgtm-com]

This pull request introduces 1 alert and fixes 5 when merging c1731b1 into b87e929 - view on LGTM.com

new alerts:

• 1 for Useless assignment to local variable

fixed alerts:

• 5 for Unused variable, import, function or class

[captainsafia]

Left some comments inline.

I tried to run a cell with these commits and ran into some issues. The Binder instance was successfully connected.

I captured some error traces from the run below.

[lgtm-com]

This pull request introduces 1 alert and fixes 5 when merging 7900efd into b09cfb5 - view on LGTM.com

new alerts:

• 1 for Useless assignment to local variable

fixed alerts:

• 5 for Unused variable, import, function or class

[lgtm-com]

This pull request introduces 1 alert and fixes 5 when merging 7f7a8d7 into b09cfb5 - view on LGTM.com

new alerts:

• 1 for Useless assignment to local variable

fixed alerts:

• 5 for Unused variable, import, function or class

[lgtm-com]

This pull request introduces 2 alerts and fixes 5 when merging dd7c305 into b09cfb5 - view on LGTM.com

new alerts:

• 1 for Useless assignment to local variable
• 1 for Unused variable, import, function or class

fixed alerts:

• 5 for Unused variable, import, function or class

[lgtm-com]

This pull request introduces 2 alerts and fixes 5 when merging ee80a60 into b09cfb5 - view on LGTM.com

new alerts:

• 1 for Useless assignment to local variable
• 1 for Unused variable, import, function or class

fixed alerts:

• 5 for Unused variable, import, function or class

[ramantehlan]

Thank you for the review @captainsafia.

While trying to reproduce the cell not running issue, I observed that It happens only when we open a notebook before it is connected to the server, so it doesn't work even after it gets connected to the server. If we reload the page, it starts to work again. I will open an issue for the same.

[captainsafia]

Merging now so we can iterate in smaller PRs.