moved getting orders for dashboard to App component

this way the orders are downloaded only once at the start of the application instead of every time when the user navigates to `/`
kjendrzyca · Mar 27, 2016 · 9ee5fb8 · 9ee5fb8
1 parent 803581e
commit 9ee5fb8
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 23 deletions.
diff --git a/src/components/app.jsx b/src/components/app.jsx
@@ -13,6 +13,7 @@ import OrderDetails from './orderDetails';
 import {orders, activeOrder} from '../store/orders';
 import localization from '../store/localization';
 import changeLocale from '../store/localizationActions';
+import {hydrateOrders} from '../store/ordersActions';
 
 // Add the reducer to your store on the `routing` key
 const store = createStore(
@@ -31,9 +32,30 @@ const App = connect(state => ({resources: state.localization.resources.app}))(
     React.createClass({
         propTypes: {
             children: React.PropTypes.object,
+            dispatch: React.PropTypes.func.isRequired,
             resources: React.PropTypes.object.isRequired
         },
 
+        componentDidMount () {
+            fetch('/api/orders')
+                .then(response => response.json())
+                .then(orders => {
+                    const ordersToday = orders.map(orderWithStringDates => {
+                        return {
+                            ...orderWithStringDates,
+                            deadline: new Date(orderWithStringDates.deadline)
+                        }
+                    }).filter(order => {
+                        const today = new Date();
+                        return today.getFullYear() === order.deadline.getFullYear() &&
+                            today.getMonth() === order.deadline.getMonth() &&
+                            today.getDate() === order.deadline.getDate();
+                    });
+
+                    this.props.dispatch(hydrateOrders(ordersToday));
+                });
+        },
+
         render () {
             return (
                 <Grid>

diff --git a/src/components/dashboard.jsx b/src/components/dashboard.jsx
@@ -3,40 +3,18 @@ import {Row, Col, Alert, Button} from 'react-bootstrap';
 import {Link} from 'react-router';
 import OrderTile from './orderTile';
 import {connect} from 'react-redux';
-import {hydrateOrders} from '../store/ordersActions';
 
 const Dashboard = React.createClass({
     propTypes: {
-        dispatch: React.PropTypes.func.isRequired,
         orders: React.PropTypes.array.isRequired,
         resources: React.PropTypes.object.isRequired
     },
 
-    componentDidMount () {
-        fetch('/api/orders')
-            .then(response => response.json())
-            .then(orders => {
-                const ordersToday = orders.map(orderWithStringDates => {
-                    return {
-                        ...orderWithStringDates,
-                        deadline: new Date(orderWithStringDates.deadline)
-                    }
-                }).filter(order => {
-                    const today = new Date();
-                    return today.getFullYear() === order.deadline.getFullYear() &&
-                        today.getMonth() === order.deadline.getMonth() &&
-                        today.getDate() === order.deadline.getDate();
-                });
-
-                this.props.dispatch(hydrateOrders(ordersToday));
-            });
-    },
-
     _renderOrderTiles () {
         return this.props.orders.map(order => <OrderTile key={order.id} {...order} />);
     },
 
-    _getTileContainerStyles() {
+    _getTileContainerStyles () {
         return {
             marginTop: '1em'
         };