Merge pull request #18 from AcademiaY4/feat/user-management

fix:hashingPassword
AcademiaY4 · Oct 19, 2024 · b709f90 · b709f90
2 parents fec21aa + f2625b4
commit b709f90
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 0 deletions.
diff --git a/Models/Rating.cs b/Models/Rating.cs
@@ -9,6 +9,16 @@ public class Rating
 {
     [Required]
     public required int Stars { get; set; } 
+
+    [Required]
+    public required int ItemQualityRating { get; set; } 
+
+    [Required]
+    public required int CommunicationRating { get; set; } 
+
+    [Required]
+    public required decimal ShippingSpeedRating { get; set; } 
+
     [Required]
     public required string Comment { get; set; }  
     [BsonRepresentation(BsonType.ObjectId)]

diff --git a/Models/User.cs b/Models/User.cs
@@ -41,4 +41,6 @@ public class User
     public DateTime DateCreated { get; set; } = DateTime.UtcNow;
     // Applicable for vendors only
     public List<Rating>? VendorRatings { get; set; } = null;
+    public string? ShopName {get;set;}
+    public string? ShopDescription {get;set;}
 }
diff --git a/Services/Impl/AdminService.cs b/Services/Impl/AdminService.cs
@@ -91,6 +91,10 @@ public async Task<ApiRes> CreateUser(CreateUserDto createUserDto)
             var newUser = _mapper.Map<User>(createUserDto);
             // Auto activate account when admin creates users
             newUser.IsApproved = true;
+            newUser.Status= Models.Enums.Status.ACTIVE;
+             // Hash the new password
+            var newPasswordHash = HashPassword.CreatePasswordHash(createUserDto.Password);
+            newUser.PasswordHash = newPasswordHash;
             newUser.Status = Models.Enums.Status.ACTIVE;
 
             await _adminRepository.CreateNewUser(newUser);