Merge pull request #170 from amida-tech/feature/GREY-252

GREY-252 org required field
amida-tech · Aug 12, 2016 · 377908f · 377908f
2 parents 73224d6 + dd8d334
commit 377908f
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/backend/app/controllers/organizations.js b/backend/app/controllers/organizations.js
@@ -454,6 +454,9 @@ function* checkOrgData(req) {
             throw new HttpError(400, 'Realm \'' + req.body.realm + '\' already exists');
         }
     } else {
+        if (!req.body.name) {
+            throw new HttpError(400, 'name field is required');
+        }
         delete req.body.realm; // do not allow to edit realm in organization
     }
 

diff --git a/client/app/greyscale.tables/scripts/services/greyscale-organizations.js b/client/app/greyscale.tables/scripts/services/greyscale-organizations.js
@@ -20,12 +20,14 @@ angular.module('greyscale.tables')
             field: 'realm',
             show: true,
             title: 'Realm',
-            dataReadOnly: 'edit'
+            dataReadOnly: 'edit',
+            dataRequired: true
         }, {
             field: 'name',
             show: true,
             sortable: 'name',
-            title: tns + 'NAME'
+            title: tns + 'NAME',
+            dataRequired: true
         }, {
             field: 'address',
             show: true,