Merge pull request #108 from plotly/remove-authentication

Remove `Authentication` component.

CHANGELOG.md

@@ -2,6 +2,10 @@
 All notable changes to this project will be documented in this file.
 This project adheres to [Semantic Versioning](http://semver.org/).
 
+## [UNRELEASED] - 2018-12-17
+### Removed
+- Unused login api and Authentication component [#108](https://github.com/plotly/dash-renderer/pull/108)
+
 ## [0.16.1] - 2018-12-14
 ### Fixed
 - Added `key` to rendered components, fixing [#379](https://github.com/plotly/dash-core-components/issues/379)

src/AppContainer.react.js

@@ -1,26 +1,37 @@
 import {connect} from 'react-redux';
 import React from 'react';
-import Authentication from './Authentication.react';
+import PropTypes from 'prop-types';
 import APIController from './APIController.react';
 import DocumentTitle from './components/core/DocumentTitle.react';
 import Loading from './components/core/Loading.react';
 import Toolbar from './components/core/Toolbar.react';
 import Reloader from './components/core/Reloader.react';
+import {readConfig} from './actions';
 
-function UnconnectedAppContainer() {
-    return (
-        <Authentication>
+class UnconnectedAppContainer extends React.Component {
+
+    componentWillMount() {
+        const {dispatch} = this.props;
+        dispatch(readConfig())
+    }
+
+    render() {
+        return (
             <div>
                 <Toolbar />
                 <APIController />
                 <DocumentTitle />
                 <Loading />
                 <Reloader />
             </div>
-        </Authentication>
-    );
+        );
+    }
 }
 
+UnconnectedAppContainer.propTypes = {
+    dispatch: PropTypes.func,
+};
+
 const AppContainer = connect(
     state => ({
         history: state.history,

src/actions/api.js

@@ -91,12 +91,6 @@ export function getDependencies() {
     return apiThunk('_dash-dependencies', 'GET', 'dependenciesRequest');
 }
 
-export function login(oauth_token) {
-    return apiThunk('_dash-login', 'POST', 'loginRequest', '', '', {
-        Authorization: `Bearer ${oauth_token}`,
-    });
-}
-
 export function getReloadHash() {
     return apiThunk('_reload-hash', 'GET', 'reloadRequest');
 }

src/components/core/Reloader.react.js

@@ -51,6 +51,7 @@ class Reloader extends React.Component {
                 ) {
                     // Look if it was a css file.
                     let was_css = false;
+                    // eslint-disable-next-line prefer-const
                     for (let a of reloadRequest.content.files) {
                         if (a.is_css) {
                             was_css = true;

src/reducers/api.js

@@ -36,5 +36,4 @@ function createApiReducer(store) {
 
 export const dependenciesRequest = createApiReducer('dependenciesRequest');
 export const layoutRequest = createApiReducer('layoutRequest');
-export const loginRequest = createApiReducer('loginRequest');
 export const reloadRequest = createApiReducer('reloadRequest');

src/reducers/reducer.js

@@ -19,7 +19,6 @@ const reducer = combineReducers({
     config,
     dependenciesRequest: API.dependenciesRequest,
     layoutRequest: API.layoutRequest,
-    loginRequest: API.loginRequest,
     reloadRequest: API.reloadRequest,
     history,
 });