Merge pull request #107 from Orange-Co/feature/common

chore: set response dto
Orange-Co · Sep 22, 2024 · c34bbb6 · c34bbb6
2 parents 5dd77a2 + e085164
commit c34bbb6
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/main/java/co/orange/ddanzi/service/auth/AuthService.java b/src/main/java/co/orange/ddanzi/service/auth/AuthService.java
@@ -77,7 +77,7 @@ public ApiResponse<?> verify(VerifyRequestDto requestDto){
 
         Authentication authentication = user.getAuthentication();
         if(authentication != null) {
-            verifyExistingAuthentication(user, requestDto);
+            return verifyExistingAuthentication(user, requestDto);
         }
         else{
             String phone = formatPhone(requestDto.getPhone());
@@ -90,8 +90,8 @@ public ApiResponse<?> verify(VerifyRequestDto requestDto){
             log.info("회원 정보 변경 완료 user_authentication_id -> {}", user.getAuthentication().getId());
 
             termService.createUserAgreements(user, requestDto.getIsAgreedMarketingTerm());
+            return ApiResponse.onSuccess(Success.CREATE_AUTHENTICATION_SUCCESS, setVerifyResponse(user));
         }
-        return ApiResponse.onSuccess(Success.CREATE_AUTHENTICATION_SUCCESS, setVerifyResponse(user));
     }
 
     @Transactional