Merge pull request #790 from Lunatic-Labs/SKIL-202

Make SuperAdmin also an admin; send null to db for lms_id is not set.
Lunatic-Labs · Nov 26, 2024 · 2f1f261 · 2f1f261
2 parents e03ba2c + 6a6ee36
commit 2f1f261
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/BackEndFlask/models/user.py b/BackEndFlask/models/user.py
@@ -243,7 +243,8 @@ def load_SuperAdminUser():
         "lms_id": 0,
         "consent": None,
         "owner_id": None,
-        "role_id": None
+        "role_id": None,
+        "is_admin": True
     })
 
 # user_id = 2

diff --git a/FrontEndReact/src/View/Admin/Add/AddUsers/AdminAddUser.js b/FrontEndReact/src/View/Admin/Add/AddUsers/AdminAddUser.js
@@ -188,7 +188,7 @@ class AdminAddUser extends Component {
             "first_name": firstName,
             "last_name": lastName,
             "email": email,
-            "lms_id": lmsId,
+            "lms_id": lmsId != "" ? lmsId : null,
             "consent": null,
             "owner_id": cookies.get('user')['user_id'],
             "role_id": navbar.props.isSuperAdmin ? 3 : role