add notification & validation messages

Author: mertkahyaoglu

docs/frontend/constants.md

@@ -11,10 +11,12 @@ Stores all of the action creators' names
 
 Stores the following messages;
 
-* delete message
-* leave message
-* override static file message
+* confirm delete message
+* confirm leave message
+* confirm override static file message
 * not found message
+* notification messages
+* validation messages
 
 This allows us to control all of the messages in a single place and also will help us localize the admin panel in the future.
 

src/actions/collections.js

@@ -1,7 +1,10 @@
 import * as ActionTypes from '../constants/actionTypes';
 import _ from 'underscore';
 import moment from 'moment';
-
+import { validationError } from '../actions/utils';
+import { get, put, del } from '../utils/fetch';
+import { validator } from '../utils/validation';
+import { slugify } from '../utils/helpers';
 import {
   getCollectionsUrl,
   getCollectionUrl,
@@ -10,18 +13,16 @@ import {
   putCollectionDocumentUrl,
   deleteCollectionDocumentUrl
 } from '../constants/api';
-
 import {
   getCollections,
   getCollection,
   getCollectionDocuments
 } from '../constants/api';
-
-import { validationError } from '../actions/utils';
-
-import { get, put, del } from '../utils/fetch';
-import { validator } from '../utils/validation';
-import { slugify } from '../utils/helpers';
+import {
+  getTitleRequiredMessage,
+  getFilenameRequiredMessage,
+  getFilenameNotValidMessage
+} from '../constants/messages';
 
 export function fetchCollections() {
   return dispatch => {
@@ -76,15 +77,15 @@ export function putDocument(id, collection) {
         'path': 'required'
       };
       let messages = {
-        'title.required': 'The title is required.',
-        'path.required': 'The filename is required.'
+        'title.required': getTitleRequiredMessage(),
+        'path.required': getFilenameRequiredMessage()
       };
       if (collection == 'posts') {
         validations['path'] = 'required|date';
-        messages['path.date'] = 'The filename is not valid.';
+        messages['path.date'] = getFilenameNotValidMessage();
       }else {
         validations['path'] = 'required|filename';
-        messages['path.filename'] = 'The filename is not valid.';
+        messages['path.filename'] = getFilenameNotValidMessage();
       }
       const errors = validator(metadata, validations, messages);
       if(errors.length) {
@@ -101,7 +102,6 @@ export function putDocument(id, collection) {
       }),
       raw_content
     });
-    // TODO dispatch({type: ActionTypes.PUT_DOCUMENT_REQUEST, doc});
     return put(
       putCollectionDocumentUrl(
         collection, id || path.substring(path.lastIndexOf('/') + 1)

src/actions/config.js

@@ -1,8 +1,7 @@
 import * as ActionTypes from '../constants/actionTypes';
 import { getConfigurationUrl, putConfigurationUrl } from '../constants/api';
-
+import { getParserErrorMessage } from '../constants/messages';
 import { addNotification } from './notifications';
-
 import { get, put } from '../utils/fetch';
 import { toJSON } from '../utils/helpers';
 
@@ -24,7 +23,7 @@ export function putConfig(config) {
     try {
       json = toJSON(config);
     } catch (e) {
-      return dispatch(addNotification('Parse Error', e.message, 'error'));
+      return dispatch(addNotification(getParserErrorMessage(), e.message, 'error'));
     }
     return put(
       putConfigurationUrl(),

src/actions/datafiles.js

@@ -1,20 +1,18 @@
 import * as ActionTypes from '../constants/actionTypes';
 import _ from 'underscore';
-
+import { getParserErrorMessage } from '../constants/messages';
+import { validationError } from './utils';
+import { addNotification } from './notifications';
+import { get, put, del } from '../utils/fetch';
+import { toJSON } from '../utils/helpers';
+import { validator } from '../utils/validation';
 import {
   getDataFilesUrl,
   getDataFileUrl,
   putDataFileUrl,
   deleteDataFileUrl
 } from '../constants/api';
 
-import { validationError } from './utils';
-import { addNotification } from './notifications';
-
-import { get, put, del } from '../utils/fetch';
-import { toJSON } from '../utils/helpers';
-import { validator } from '../utils/validation';
-
 export function fetchDataFiles() {
   return (dispatch) => {
     dispatch({ type: ActionTypes.FETCH_DATAFILES_REQUEST});
@@ -60,7 +58,7 @@ export function putDataFile(filename, data) {
     try {
       json = toJSON(data);
     } catch (e) {
-      return dispatch(addNotification('Parse Error', e.message, 'error'));
+      return dispatch(addNotification(getParserErrorMessage(), e.message, 'error'));
     }
     return put(
       putDataFileUrl(filename),

src/actions/pages.js

@@ -1,18 +1,20 @@
 import * as ActionTypes from '../constants/actionTypes';
 import _ from 'underscore';
-
+import { validationError } from '../actions/utils';
+import { get, put, del } from '../utils/fetch';
+import { validator } from '../utils/validation';
+import { slugify } from '../utils/helpers';
 import {
   getPagesUrl,
   getPageUrl,
   putPageUrl,
   deletePageUrl
 } from '../constants/api';
-
-import { validationError } from '../actions/utils';
-
-import { get, put, del } from '../utils/fetch';
-import { validator } from '../utils/validation';
-import { slugify } from '../utils/helpers';
+import {
+  getTitleRequiredMessage,
+  getFilenameRequiredMessage,
+  getFilenameNotValidMessage
+} from '../constants/messages';
 
 export function fetchPages() {
   return (dispatch) => {
@@ -49,8 +51,8 @@ export function putPage(name) {
         metadata,
         { 'path': 'required|filename' },
         {
-          'path.required': 'The filename is required.',
-          'path.filename': 'The filename is not valid.'
+          'path.required': getTitleRequiredMessage(),
+          'path.filename': getFilenameNotValidMessage()
         }
       );
       if (errors.length) {
@@ -67,7 +69,6 @@ export function putPage(name) {
       }),
       raw_content
     });
-    // TODO dispatch({type: ActionTypes.PUT_PAGE_REQUEST, page});
     return put(
       putPageUrl(name || path),
       page,

src/actions/staticfiles.js

@@ -1,17 +1,20 @@
 import * as ActionTypes from '../constants/actionTypes';
 import _ from 'underscore';
-
+import { get, put, del } from '../utils/fetch';
+import { addNotification } from './notifications';
+import {
+  getSuccessMessage,
+  getErrorMessage,
+  getUploadSuccessMessage,
+  getUploadErrorMessage
+} from '../constants/messages';
 import {
   getStaticFilesUrl,
   getStaticFileUrl,
   putStaticFileUrl,
   deleteStaticFileUrl
 } from '../constants/api';
 
-import { get, put, del } from '../utils/fetch';
-
-import { addNotification } from './notifications';
-
 export function fetchStaticFiles() {
   return dispatch => {
     dispatch({ type: ActionTypes.FETCH_STATICFILES_REQUEST});
@@ -47,8 +50,8 @@ export function uploadStaticFiles(files) {
           });
           dispatch(fetchStaticFiles());
           dispatch(addNotification(
-            'Success',
-            `${file.name} uploaded successfully`,
+            getSuccessMessage(),
+            getUploadSuccessMessage(file.name),
             'success'
           ));
         })
@@ -58,8 +61,8 @@ export function uploadStaticFiles(files) {
             error
           });
           dispatch(addNotification(
-            'Upload Error',
-            `Error occurred uploading the file.`,
+            getErrorMessage(),
+            getUploadErrorMessage(),
             'error'
           ));
         });

src/constants/messages.js

@@ -1,3 +1,4 @@
+// inline messages
 export const getDeleteMessage = (filename) =>
   `Are you sure that you want to delete "${filename}" ?`;
 
@@ -9,3 +10,35 @@ export const getNotFoundMessage = (type) =>
 
 export const getOverrideMessage = (filename) =>
   `${filename} will be overwritten. Continue anyway?`;
+
+// notification messages
+export const getParserErrorMessage = () => "Parse Error";
+
+export const getSuccessMessage = () => "Success";
+
+export const getErrorMessage = () => "Error";
+
+export const getUploadSuccessMessage = (filename) =>
+  `${filename} uploaded successfully`;
+
+export const getUploadErrorMessage = () =>
+  `Error occurred uploading the file.`;
+
+export const getFetchErrorMessage = (filename) =>
+  `Could not fetch the ${filename}`;
+
+export const getUpdateErrorMessage = (filename) =>
+  `Could not update the ${filename}`;
+
+export const getDeleteErrorMessage = (filename) =>
+  `Could not delete the ${filename}`;
+
+// validation messages
+export const getTitleRequiredMessage = () =>
+  "The title is required.";
+
+export const getFilenameRequiredMessage = () =>
+  "The filename is required.";
+
+export const getFilenameNotValidMessage = () =>
+  "The filename is not valid.";

src/utils/fetch.js

@@ -1,6 +1,11 @@
 import fetch from 'isomorphic-fetch';
-
 import { addNotification } from '../actions/notifications';
+import {
+  getErrorMessage,
+  getFetchErrorMessage,
+  getUpdateErrorMessage,
+  getDeleteMessage
+} from '../constants/messages';
 
 /**
  * Fetch wrapper for GET request that dispatches actions according to the
@@ -23,8 +28,8 @@ export const get = (url, action_success, action_failure, dispatch) => {
         [action_failure.name]: error
       });
       dispatch(addNotification(
-        'Fetch Error',
-        `Could not fetch the ${action_success.name}`,
+        getErrorMessage(),
+        getFetchErrorMessage(action_success.name),
         'error'
       ));
     });
@@ -59,8 +64,8 @@ export const put = (url, body, action_success, action_failure, dispatch) => {
       [action_failure.name]: error
     });
     dispatch(addNotification(
-      'Update Error',
-      `Could not update the ${action_success.name}`,
+      getErrorMessage(),
+      getUpdateErrorMessage(action_success.name),
       'error'
     ));
   });
@@ -88,8 +93,8 @@ export const del = (url, action_success, action_failure, dispatch) => {
       [action_failure.name]: error
     });
     dispatch(addNotification(
-      'Delete Error',
-      `Could not delete the ${action_success.name}`,
+      getErrorMessage(),
+      getDeleteMessage(action_success.name),
       'error'
     ));
   });