[refactor-1] separate actionCreators:

ungitConfig: for ungit configuraion
userConfig: for user configuration
version: for the versions of ungit itself and git
common: some common actionCreators
bootstrap: for initialization, aggregrate some actionCreators and use it when app init

Author: AllenFang

src-react/actions/bootstrap.js

@@ -0,0 +1,12 @@
+import { fetchUngitConfig } from './ungit-config';
+import { fetchLatestVersion, fetchGitVersion } from './version';
+import { pending } from './common';
+
+export function bootstrap() {
+  return dispatch => {
+    dispatch(pending());
+    dispatch(fetchUngitConfig());
+    dispatch(fetchLatestVersion());
+    dispatch(fetchGitVersion());
+  };
+}

src-react/actions/common.js

@@ -0,0 +1,15 @@
+/* This export common using actionCreator */
+import * as types from 'constants/action-types';
+
+export function pending() {
+  return {
+    type: types.PATH_PAGE_PENDING
+  };
+};
+
+export function apiError(message) {
+  return {
+    type: types.PATH_PAGE_API_ERR,
+    payload: message
+  };
+}

src-react/actions/ungit-config.js

@@ -1,13 +1,17 @@
 import * as types from 'constants/action-types';
+import { fetchUserConfig } from './user-config';
+import { apiError } from './common';
 
 export function fetchUngitConfig() {
   return dispatch => {
-    dispatch(pending());
     // consider wrap API call in separate modules
     // it will be easy to stub module's function when testing
     fetch('http://localhost:8448/ungit/config')
       .then(response => response.json())
       .then(json => {
+        if (!json.config.bugtracking) {
+          dispatch(fetchUserConfig());
+        }
         dispatch(receiveUgitConfig(json));
       })
       .catch(e => {
@@ -16,22 +20,9 @@ export function fetchUngitConfig() {
   };
 };
 
-export function receiveUgitConfig(ungitConfig) {
+function receiveUgitConfig(ungitConfig) {
   return {
     type: types.RECEIVE_UNGIT_CONFIG,
     payload: ungitConfig
   };
-};
-
-export function pending() {
-  return {
-    type: types.PATH_PAGE_PENDING
-  };
-};
-
-export function apiError(message) {
-  return {
-    type: types.PATH_PAGE_API_ERR,
-    payload: message
-  };
-}
+};

src-react/actions/user-config.js

@@ -0,0 +1,25 @@
+import * as types from 'constants/action-types';
+import { apiError } from './common';
+
+export function fetchUserConfig() {
+  return dispatch => {
+    // consider wrap API call in separate modules
+    // it will be easy to stub module's function when testing
+    fetch('http://localhost:8448/api/userconfig')
+      .then(response => response.json())
+      .then(json => {
+        dispatch(receiveUserConfig(json));
+      })
+      .catch(e => {
+        dispatch(apiError(e.message));
+      });
+  };
+};
+
+
+function receiveUserConfig(userConfig) {
+  return {
+    type: types.RECEIVE_USER_CONFIG,
+    payload: userConfig
+  };
+};

src-react/actions/version.js

@@ -0,0 +1,46 @@
+import * as types from 'constants/action-types';
+import { apiError } from './common';
+
+export function fetchLatestVersion() {
+  return dispatch => {
+    // consider wrap API call in separate modules
+    // it will be easy to stub module's function when testing
+    fetch('http://localhost:8448/api/latestversion')
+      .then(response => response.json())
+      .then(json => {
+        dispatch(receiveLatestVersion(json));
+      })
+      .catch(e => {
+        dispatch(apiError(e.message));
+      });
+  };
+}
+
+export function fetchGitVersion() {
+  return dispatch => {
+    // consider wrap API call in separate modules
+    // it will be easy to stub module's function when testing
+    fetch('http://localhost:8448/api/gitversion')
+      .then(response => response.json())
+      .then(json => {
+        dispatch(receiveGitVersion(json));
+      })
+      .catch(e => {
+        dispatch(apiError(e.message));
+      });
+  };
+}
+
+function receiveGitVersion(gitVersion) {
+  return {
+    type: types.RECEIVE_GIT_VERSION,
+    payload: gitVersion
+  };
+}
+
+function receiveLatestVersion(latestVersion) {
+  return {
+    type: types.RECEIVE_LATEST_VERSION,
+    payload: latestVersion
+  };
+};

src-react/constants/action-types.js

@@ -1,3 +1,6 @@
+export const RECEIVE_GIT_VERSION = 'RECEIVE_GIT_VERSION';
+export const RECEIVE_LATEST_VERSION = 'RECEIVE_LATEST_VERSION';
+export const RECEIVE_USER_CONFIG = 'RECEIVE_USER_CONFIG';
 export const RECEIVE_UNGIT_CONFIG = 'RECEIVE_UNGIT_CONFIG';
 export const PATH_PAGE_PENDING = 'PATH_PAGE_PENDING';
 export const PATH_PAGE_API_ERR = 'PATH_PAGE_API_ERR';