Improve the way create user button is handled. Set connection cutoff to its final value

Author: ubenzer

client/containers/Users/Users.jsx

@@ -15,6 +15,7 @@ import './Users.styles.css';
 class Users extends Component {
   static propTypes = {
     loading: PropTypes.bool.isRequired,
+    connectionsLoading: PropTypes.bool.isRequired,
     error: PropTypes.string,
     users: PropTypes.array,
     connections: PropTypes.array,
@@ -44,7 +45,9 @@ class Users extends Component {
 
   componentWillMount = () => {
     this.props.fetchUsers();
-    this.props.fetchConnections();
+    if (!this.props.connectionsLoading) {
+      this.props.fetchConnections();
+    }
   };
 
   onPageChange = (page) => {
@@ -77,7 +80,7 @@ class Users extends Component {
       error,
       users,
       total,
-      connections,
+      connectionsLoading,
       accessLevel,
       nextPage,
       pages,
@@ -102,7 +105,7 @@ class Users extends Component {
         <div className="row content-header">
           <div className="col-xs-12 user-table-content">
             <h1>{languageDictionary.usersTitle || 'Users'}</h1>
-            {( role > 0 && showCreateUser) ?
+            {( !connectionsLoading && role > 0 && showCreateUser) ?
               <button id="create-user-button" className="btn btn-success pull-right new" onClick={this.createUser}>
                 <i className="icon-budicon-473"></i>
                 {languageDictionary.createUserButtonText || 'Create User'}
@@ -158,6 +161,7 @@ function mapStateToProps(state) {
     loading: state.users.get('loading'),
     users: state.users.get('records').toJS(),
     connections: state.connections.get('records').toJS(),
+    connectionsLoading: state.connections.get('loading'),
     total: state.users.get('total'),
     nextPage: state.users.get('nextPage'),
     pages: state.users.get('pages'),

server/lib/multipartRequest.js

@@ -21,8 +21,6 @@ export default function(client, entity, opts = {}, fetchOptions = {} ) {
   let total = 0;
   let pageCount = 0;
 
-  console.log({options})
-
   const getTotals = () =>
     getter({ ...options, include_totals: true, page: 0 })
       .then((response) => {

server/routes/connections.js

@@ -6,8 +6,8 @@ import multipartRequest from '../lib/multipartRequest';
 // This is the number of connections in a tenant which the DAE can reasonably handle. More than this and it fails to
 // finish loading connections and the "create user" button is never shown. If there are more connections than this
 // in the tenant, we will return zero connections to the front end, and it will use a free text box for connection name
-// in the create user dialogue. 
-const CONNECTIONS_FETCH_LIMIT = 200;
+// in the create user dialogue.
+const CONNECTIONS_FETCH_LIMIT = 20000;
 
 export default (scriptManager) => {
   const api = Router();
@@ -16,7 +16,7 @@ export default (scriptManager) => {
       req.auth0,
       'connections',
       { strategy: 'auth0', fields: 'id,name,strategy,options' },
-      { limit: CONNECTIONS_FETCH_LIMIT, perPage: 20 }
+      { limit: CONNECTIONS_FETCH_LIMIT, perPage: 100 }
     )
       .then((connections) => {
         global.connections = connections.map(conn => ({ name: conn.name, id: conn.id }));