Merge pull request #136 from loeck/master

Don't sync if we don't have accounts

src/actions/accounts.js

@@ -6,6 +6,8 @@ import sortBy from 'lodash/sortBy'
 import type { Dispatch } from 'redux'
 import type { Account } from 'types/common'
 
+import { startSyncAccounts } from 'renderer/events'
+
 function sortAccounts(accounts, orderAccounts) {
   const accountsSorted = sortBy(accounts, a => {
     if (orderAccounts === 'balance') {
@@ -36,12 +38,17 @@ export const updateOrderAccounts: UpdateOrderAccounts = (orderAccounts: string)
 
 export type AddAccount = Account => (Function, Function) => void
 export const addAccount: AddAccount = payload => (dispatch, getState) => {
-  const { settings: { orderAccounts } } = getState()
+  const { settings: { orderAccounts }, accounts } = getState()
   dispatch({
     type: 'ADD_ACCOUNT',
     payload,
   })
   dispatch(updateOrderAccounts(orderAccounts))
+
+  // Start sync accounts the first time you add an account
+  if (accounts.length === 0) {
+    startSyncAccounts([payload])
+  }
 }
 
 export type RemoveAccount = Account => { type: string, payload: Account }

src/index.ejs

@@ -45,7 +45,7 @@
         position: fixed;
         right: 0;
         top: 0;
-        transition: opacity 0.6s ease-in-out;
+        transition: opacity 0.4s ease-in-out;
         z-index: 100;
       }
       #preload video {
@@ -81,12 +81,19 @@
       const preloadEl = document.getElementById('preload')
       const appEl = document.getElementById('app')
 
-      const initApp = () => {
+      const initApp = (options = {}) => {
+        const { force = false } = options
+
         appEl.style.display = 'flex'
+
+        if (force) {
+          preloadEl.remove()
+        } else {
           preloadEl.style.opacity = 0
 
           preloadEl.addEventListener('transitionend', () => preloadEl.remove())
         }
+      }
 
       if (name === 'MainWindow') {
         preloadEl.style.display = 'flex'
@@ -103,7 +110,7 @@
           setTimeout(initApp, delay > 0 ? delay : 1)
         }
       } else {
-        initApp()
+        initApp({ force: true })
       }
     </script>
   </body>

src/renderer/events.js

@@ -179,9 +179,11 @@ export default ({ store, locked }: { store: Object, locked: boolean }) => {
   if (!locked && !DISABLED_SYNC) {
     const accounts = getAccounts(store.getState())
 
-    // Start accounts sync
+    // Start accounts sync only if we have accounts
+    if (accounts.length > 0) {
       startSyncAccounts(accounts)
     }
+  }
 
   if (__PROD__) {
     // Start check of eventual updates