Merge pull request #10 from PolkadotEducation/artur-isAdmin-fix

Remove isAdmin from createUser() (POST /users)
PolkadotEducation · Aug 29, 2024 · 799d60a · 799d60a
2 parents 6415e1a + da85c97
commit 799d60a
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/controllers/users.ts b/src/controllers/users.ts
@@ -4,10 +4,10 @@ import { sendVerificationEmail } from "@/helpers/aws/ses";
 
 export const createUser = async (req: Request, res: Response) => {
   try {
-    const { email, password, name, company, isAdmin } = req.body;
+    const { email, password, name, company } = req.body;
     if (!email || !password) return res.status(400).send({ error: { message: "Missing email or password" } });
 
-    const newUser = await UserModel.createUser(email, password, name, company, isAdmin);
+    const newUser = await UserModel.createUser(email, password, name, company);
     if (newUser) {
       await sendVerificationEmail(email, newUser.verifyToken!);
       newUser.verifyToken = undefined;

diff --git a/src/models/User.ts b/src/models/User.ts
@@ -46,7 +46,7 @@ class User extends BaseModel {
     password: string,
     name: string,
     company: string,
-    isAdmin: boolean,
+    isAdmin?: boolean,
   ): Promise<UserInfo | undefined> {
     try {
       const exists = await this.findOne({ email: email.toLowerCase() });