diff --git a/src/main/java/com/skillsync/skillsync/controller/UserController.java b/src/main/java/com/skillsync/skillsync/controller/UserController.java
index 052359c..b23ff79 100644
--- a/src/main/java/com/skillsync/skillsync/controller/UserController.java
+++ b/src/main/java/com/skillsync/skillsync/controller/UserController.java
@@ -1,6 +1,7 @@
 package com.skillsync.skillsync.controller;
 
 import org.springframework.beans.factory.annotation.Autowired;
+import org.springframework.http.ResponseEntity;
 import org.springframework.web.bind.annotation.*;
 
 import com.skillsync.skillsync.dto.UserUpdateDTO;
@@ -8,6 +9,7 @@
 import com.skillsync.skillsync.model.User;
 import com.skillsync.skillsync.service.SkillService;
 import com.skillsync.skillsync.service.UserService;
+import com.skillsync.skillsync.service.impl.UserServiceImpl;
 
 import java.util.*;
 
@@ -15,18 +17,24 @@
 @RequestMapping("/api/users")
 public class UserController {
 
+    @Autowired
+    UserServiceImpl userServiceImpl;
+
+    public UserController(UserServiceImpl userServiceImpl){
+        this.userServiceImpl = userServiceImpl;
+    }
+
     // Save user
     @PostMapping("/register")
     public User saveUser(@RequestBody User user) {
-        // TODO: Add logic to save user
-        return null;
+        return userServiceImpl.saveUser(user);
     }
 
     //get all users
     @GetMapping("/allusers")
     public List<User> getAllUsers() {
         // TODO: Add logic to fetch all users
-        return null;
+        return userServiceImpl.getAllUsers();
     }
 
     // get user by id
diff --git a/src/main/java/com/skillsync/skillsync/dto/UserUpdateDTO.java b/src/main/java/com/skillsync/skillsync/dto/UserUpdateDTO.java
index 4d14986..8b7a347 100644
--- a/src/main/java/com/skillsync/skillsync/dto/UserUpdateDTO.java
+++ b/src/main/java/com/skillsync/skillsync/dto/UserUpdateDTO.java
@@ -1,8 +1,20 @@
 package com.skillsync.skillsync.dto;
 
+import lombok.AllArgsConstructor;
+import lombok.Data;
+import lombok.NoArgsConstructor;
+
+@Data
+@AllArgsConstructor
+@NoArgsConstructor
 public class UserUpdateDTO {
 
     // Add feilds to update user
     // this file helps not to senf all the fields of user entity to update
+    private Long id;
+    private String email;
+    private String bio;
+    private String roleType;
+    private boolean isAvailableForMentorship;
 
 }