Merge pull request #62 from nikhilnarayanan623/fix/sign-up-verify

user sign-up verify database update issue fixed

makefile

@@ -13,11 +13,12 @@ ${BINARY_DIR}:
 	mkdir -p $(BINARY_DIR)
 
 build: ${BINARY_DIR} ## Compile the code, build Executable File
-#	$(GOCMD) build -o $(BINARY_DIR) -v ./cmd/api
-	GOARCH=amd64 $(GOCMD) build -v -o $(BINARY_DIR)/api-linux-amd64 ./cmd/api/main.go
+	$(GOCMD) build -o $(BINARY_DIR) -v ./cmd/api
+# 	GOARCH=amd64 $(GOCMD) build -o $(BINARY_DIR)/api-linux-amd64 -v ./cmd/api/main.go
 
 build-run: build ## run project build file if not exist build it
-	./$(BINARY_DIR)/api-linux-amd64
+#	./$(BINARY_DIR)/api-linux-amd64
+	./$(BINARY_DIR)/api
 
 run: ## Start application
 	$(GOCMD) run ./cmd/api/main.go

pkg/repository/user.go

@@ -69,7 +69,7 @@ func (c *userDatabase) SaveUser(ctx context.Context, user domain.User) (userID u
 
 	createdAt := time.Now()
 	err = c.DB.Raw(query, user.UserName, user.FirstName, user.LastName,
-		user.Age, user.Email, user.Phone, user.Password, user.GoogleImage, createdAt).Scan(&user).Error
+		user.Age, user.Email, user.Phone, user.Password, user.GoogleImage, createdAt).Scan(&userID).Error
 
 	return userID, err
 }

pkg/usecase/auth.go

@@ -179,12 +179,6 @@ func (c *authUseCase) LoginOtpVerify(ctx context.Context, otpVerifyDetails reque
 		return 0, ErrInvalidOtp
 	}
 
-	if err := c.userRepo.UpdateVerified(ctx, otpSession.UserID); err != nil {
-
-		return 0, utils.PrependMessageToError(err, "failed to update user verified in db")
-
-	}
-
 	return otpSession.UserID, nil
 }
 
@@ -378,7 +372,7 @@ func (c *authUseCase) SingUpOtpVerify(ctx context.Context,
 		return 0, ErrInvalidOtp
 	}
 
-	err = c.userRepo.UpdateVerified(ctx, otpSession.ID)
+	err = c.userRepo.UpdateVerified(ctx, otpSession.UserID)
 	if err != nil {
 		return 0, utils.PrependMessageToError(err, "failed to update user verified on database")
 	}