release: deploy to prod (#2)

* fix: 누적 잔액이 정상 반영되도록 한다. (#75) * refactor: LedgerService 메소드 분리 * refactor: move updateTotalBalance method to Entity * refactor: change calculator return type to int * refactor: 누적 잔고가 정상 반영되도록 한다. * feat: kakao 로그인 예외 처리를 세분화한다. (#76) * feat: 장부 내역 조회 api v2를 추가한다. (#77) * feat: 내정보 조회 api에 provider field를 추가한다. * feat: 특정 기간의 장부를 반환하는 메소드를 추가한다. * feat: LedgerReader의 공통 코드를 메소드 분리한다. * feat: LedgerControllerV2, RequestV2를 추가한다. * config: swagger scan 패키지를 수정한다. (#79) * feat: swagger scan 패키지를 수정한다. * feat: 누락된 Tag, Operation을 추가한다. * feat: Apple OAuth Login을 추가한다. (#80) * chore: remove unused util class * feat: OAuthProvider에 Apple을 추가한다. * feat: bouncy castle, JWT decode 라이브러리를 추가한다. * feat: yml에 apple 환경 변수를 추가한다. * feat: apple login service를 구현한다. * refactor: Apple login 시 name을 http body로 받는다. * feat: OAuth 계정을 revoke하는 메소드를 추가한다. * feat: 최초 로그인 시 authorizationCode로 refreshToken을 취득한다. * feat: AppleUser 정보를 저장하는 Entity를 추가한다. * feat: 회원가입 시 Apple User인 경우 AppleUser를 저장한다. * feat: Apple 회원 탈퇴를 구현한다. * feat: 회원탈퇴 api v2를 추가한다. * chore: CI build가 실패하지 않도록 한다. * feat: 소속 조회 결과에 소속 생성일자를 포함한다. (#82) * feat: 내 소속 조회 결과에 소속의 생성일자도 포함한다. * chore: remove unused fields * chore: userToken 필드를 삭제한다. * feat: 소속 조회 결과에 소속 생성일자를 포함한다. (#82) * feat: 내 소속 조회 결과에 소속의 생성일자도 포함한다. * chore: remove unused fields * chore: userToken 필드를 삭제한다. * chore: log를 추가한다. * chore: apple revoke url를 변경한다. * refactor: 회원 탈퇴 시 AppleUser 데이터를 hardDelete한다. * remove: 회원 탈퇴 v2 api를 삭제한다. * refactor: 장부 상세내역 CUD 리팩토링 * fix: 장부 조회 시 endMonth의 기간도 포함한다. * chore: github workflow를 재설정한다. * chore: resolve merge conflict
MONEYMONG · May 26, 2024 · 09e0fe7 · 09e0fe7
1 parent 2a8641f
commit 09e0fe7
Show file tree

Hide file tree

Showing 19 changed files with 176 additions and 131 deletions.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -2,7 +2,10 @@ name: gradle build
 
 on:
   pull_request:
-    branches: [ main ]
+    branches:
+    - main
+    - dev
+
 
 # 권한 설정
 permissions: write-all
@@ -25,10 +28,6 @@ jobs:
       - name: Set up Environment
         run: echo "${{ secrets.ENV_PROPERTIES_LOCAL }}" > ./.env
 
-      # docker로 Test용 mysql 띄우기
-      - name: Create Mysql Docker Container
-        run: sudo docker run -d -p 3306:3306 --env MYSQL_DATABASE="${{ secrets.TEST_DATABASE }}" --env MYSQL_ROOT_PASSWORD="${{ secrets.TEST_DATASOURCE_PASSWORD }}" mysql:8.0.31
-
       # 빌드시 캐시 적용
       - name: Cache Gradle Packages
         uses: actions/cache@v3

diff --git a/.github/workflows/deploy-dev.yml b/.github/workflows/deploy-dev.yml
@@ -30,16 +30,12 @@ jobs:
       - name: Send env file
         uses: appleboy/scp-action@master
         with:
-          username: ${{ secrets.USERNAME }}
-          host: ${{ secrets.NCP_HOST }}
+          username: root
+          host: ${{ secrets.NCP_DEV_IP }}
           password: ${{ secrets.NCP_PASSWORD }}
           source: "./.env"
           target: "/home/ubuntu"
 
-      # 도커 MYSQL 이미지 실행
-      - name: Create Mysql Docker Container
-        run: sudo docker run -d -p 3306:3306 --env MYSQL_DATABASE="${{ secrets.TEST_DATABASE }}" --env MYSQL_ROOT_PASSWORD="${{ secrets.TEST_DATASOURCE_PASSWORD }}" mysql:8.0.31
-
       # 빌드
       - name: Build with Gradle
         run: ./gradlew clean bootJar
@@ -55,8 +51,8 @@ jobs:
       - name: Send docker-compose.yml
         uses: appleboy/scp-action@master
         with:
-          username: ${{ secrets.USERNAME }}
-          host: ${{ secrets.NCP_HOST }}
+          username: root
+          host: ${{ secrets.NCP_DEV_IP }}
           password: ${{ secrets.NCP_PASSWORD }}
           port: 22
           source: "./.docker/docker-compose.yml"
@@ -66,8 +62,8 @@ jobs:
       - name: Deploy to Dev
         uses: appleboy/ssh-action@master
         with:
-          username: ${{ secrets.USERNAME }}
-          host: ${{ secrets.NCP_HOST }}
+          username: root
+          host: ${{ secrets.NCP_DEV_IP }}
           password: ${{ secrets.NCP_PASSWORD }}
           script: |
             sudo cp /home/ubuntu/.docker/docker-compose.yml /home/ubuntu

diff --git a/.github/workflows/deploy-prod.yml b/.github/workflows/deploy-prod.yml
@@ -30,16 +30,12 @@ jobs:
       - name: Send env file
         uses: appleboy/scp-action@master
         with:
-          username: ${{ secrets.USERNAME }}
-          host: ${{ secrets.NCP_PROD_HOST }}
+          username: root
+          host: ${{ secrets.NCP_PROD_IP }}
           password: ${{ secrets.NCP_PROD_PASSWORD }}
           source: "./.env"
           target: "/home/ubuntu"
 
-      # 도커 MYSQL 이미지 실행
-      - name: Create Mysql Docker Container
-        run: sudo docker run -d -p 3306:3306 --env MYSQL_DATABASE="${{ secrets.TEST_DATABASE }}" --env MYSQL_ROOT_PASSWORD="${{ secrets.TEST_DATASOURCE_PASSWORD }}" mysql:8.0.31
-
       # 빌드
       - name: Build with Gradle
         run: ./gradlew clean bootJar
@@ -55,19 +51,19 @@ jobs:
       - name: Send docker-compose.yml
         uses: appleboy/scp-action@master
         with:
-          username: ${{ secrets.USERNAME }}
-          host: ${{ secrets.NCP_PROD_HOST }}
+          username: root
+          host: ${{ secrets.NCP_PROD_IP }}
           password: ${{ secrets.NCP_PROD_PASSWORD }}
           port: 22
           source: "./.docker/docker-compose.yml"
           target: "/home/ubuntu/"
 
       # 도커 컴포즈 실행
-      - name: Deploy to Dev
+      - name: Deploy to Prod
         uses: appleboy/ssh-action@master
         with:
-          username: ${{ secrets.USERNAME }}
-          host: ${{ secrets.NCP_PROD_HOST }}
+          username: root
+          host: ${{ secrets.NCP_PROD_IP }}
           password: ${{ secrets.NCP_PROD_PASSWORD }}
           script: |
             sudo cp /home/ubuntu/.docker/docker-compose.yml /home/ubuntu

diff --git a/src/main/java/com/moneymong/domain/ledger/api/request/UpdateLedgerRequest.java b/src/main/java/com/moneymong/domain/ledger/api/request/UpdateLedgerRequest.java
@@ -6,7 +6,6 @@
 import java.time.ZonedDateTime;
 import lombok.Getter;
 import lombok.NoArgsConstructor;
-import org.hibernate.validator.constraints.Length;
 
 @Getter
 @NoArgsConstructor

diff --git a/src/main/java/com/moneymong/domain/ledger/entity/LedgerDetail.java b/src/main/java/com/moneymong/domain/ledger/entity/LedgerDetail.java
@@ -84,6 +84,18 @@ public void updateBalance(int balance) {
         this.balance = balance;
     }
 
+    public void updateLedgerDetailInfo(
+            String storeInfo,
+            int amount,
+            String description,
+            ZonedDateTime paymentDate
+    ) {
+        this.storeInfo = storeInfo;
+        this.amount = amount;
+        this.description = description;
+        this.paymentDate = paymentDate;
+    }
+
     public static LedgerDetail of(
             final Ledger ledger,
             final User user,

diff --git a/src/main/java/com/moneymong/domain/ledger/repository/LedgerDetailRepository.java b/src/main/java/com/moneymong/domain/ledger/repository/LedgerDetailRepository.java
@@ -4,6 +4,10 @@
 import com.moneymong.domain.ledger.entity.LedgerDetail;
 import org.springframework.data.jpa.repository.JpaRepository;
 
+import java.util.List;
+
 public interface LedgerDetailRepository extends JpaRepository<LedgerDetail, Long>, LedgerDetailCustom {
     boolean existsByLedger(Ledger ledger);
+
+    List<LedgerDetail> findAllByLedger(Ledger ledger);
 }
diff --git a/src/main/java/com/moneymong/domain/ledger/repository/impl/LedgerDetailCustomImpl.java b/src/main/java/com/moneymong/domain/ledger/repository/impl/LedgerDetailCustomImpl.java
@@ -7,7 +7,10 @@
 import com.moneymong.domain.ledger.entity.enums.FundType;
 import com.moneymong.domain.ledger.repository.LedgerDetailCustom;
 import com.querydsl.jpa.impl.JPAQueryFactory;
+
+import java.time.LocalTime;
 import java.time.ZonedDateTime;
+import java.time.temporal.TemporalAdjusters;
 import java.util.List;
 import java.util.Optional;
 
@@ -62,9 +65,12 @@ public List<LedgerDetail> searchByPeriod(
             ZonedDateTime to,
             PageRequest pageable
     ) {
+        ZonedDateTime endOfMonth = to.with(TemporalAdjusters.lastDayOfMonth())
+                .with(LocalTime.MAX);
+
         return jpaQueryFactory.selectFrom(ledgerDetail)
                 .where(ledgerDetail.ledger.eq(ledger))
-                .where(ledgerDetail.paymentDate.between(from, to))
+                .where(ledgerDetail.paymentDate.between(from, endOfMonth))
                 .orderBy(ledgerDetail.paymentDate.desc())
                 .offset((long) pageable.getPageNumber() * pageable.getPageSize())
                 .limit(pageable.getPageSize())
@@ -79,10 +85,13 @@ public List<LedgerDetail> searchByPeriodAndFundType(
             FundType fundType,
             PageRequest pageable
     ) {
+        ZonedDateTime endOfMonth = to.with(TemporalAdjusters.lastDayOfMonth())
+                .with(LocalTime.MAX);
+
         return jpaQueryFactory.selectFrom(ledgerDetail)
                 .where(ledgerDetail.ledger.eq(ledger))
                 .where(ledgerDetail.fundType.eq(fundType))
-                .where(ledgerDetail.paymentDate.between(from, to))
+                .where(ledgerDetail.paymentDate.between(from, endOfMonth))
                 .orderBy(ledgerDetail.paymentDate.desc())
                 .offset((long) pageable.getPageNumber() * pageable.getPageSize())
                 .limit(pageable.getPageSize())

diff --git a/src/main/java/com/moneymong/domain/ledger/service/manager/LedgerDetailService.java b/src/main/java/com/moneymong/domain/ledger/service/manager/LedgerDetailService.java
@@ -13,6 +13,7 @@
 import com.moneymong.domain.ledger.repository.LedgerDetailRepository;
 import com.moneymong.domain.ledger.repository.LedgerDocumentRepository;
 import com.moneymong.domain.ledger.repository.LedgerReceiptRepository;
+import com.moneymong.domain.ledger.repository.LedgerRepository;
 import com.moneymong.domain.ledger.service.mapper.LedgerAssembler;
 import com.moneymong.domain.ledger.service.reader.LedgerDocumentReader;
 import com.moneymong.domain.ledger.service.reader.LedgerReceiptReader;
@@ -22,19 +23,22 @@
 import com.moneymong.global.exception.custom.NotFoundException;
 import com.moneymong.global.exception.enums.ErrorCode;
 import java.time.ZonedDateTime;
+import java.util.Comparator;
 import java.util.List;
-import java.util.Optional;
 
 import com.moneymong.utils.AmountCalculatorByFundType;
 import lombok.RequiredArgsConstructor;
 import lombok.extern.slf4j.Slf4j;
 import org.springframework.stereotype.Service;
 import org.springframework.transaction.annotation.Transactional;
 
+import static com.moneymong.domain.ledger.entity.enums.FundType.INCOME;
+
 @Service
 @Slf4j
 @RequiredArgsConstructor
 public class LedgerDetailService {
+    private final LedgerRepository ledgerRepository;
     private final LedgerAssembler ledgerAssembler;
     private final LedgerReceiptReader ledgerReceiptReader;
     private final LedgerDocumentReader ledgerDocumentReader;
@@ -69,51 +73,44 @@ public LedgerDetail createLedgerDetail(
 
         int newAmount = AmountCalculatorByFundType.calculate(fundType, amount);
 
-        /**
-         * 가장 오래된 장부 내역인 경우 잔고를 amount 값으로 설정한다.
-         * 이전 내역이 있는 경우, 가장 가까운 시일에 생성된 장부 내역을 기준으로 잔고를 저장한다.
-         */
-        Optional<LedgerDetail> mostRecentLedgerDetail = ledgerDetailRepository.findMostRecentLedgerDetail(ledger, paymentDate);
-
-        if (mostRecentLedgerDetail.isPresent()) {
-            LedgerDetail recentDetail = mostRecentLedgerDetail.get();
-
-            int newBalance = recentDetail.getBalance() + newAmount;
-            ledgerDetail.updateBalance(newBalance);
-        }else {
-            ledgerDetail.updateBalance(newAmount);
-        }
-
         ledger.updateTotalBalance(newAmount);
 
-        ledgerDetailRepository.bulkUpdateLedgerDetailBalance(
-                ledger,
-                paymentDate,
-                newAmount
-        );
+        ledgerDetailRepository.save(ledgerDetail);
 
-        return ledgerDetailRepository.save(ledgerDetail);
+        updateBalance(ledger);
+
+        return ledgerDetail;
     }
 
     @Transactional
     public LedgerDetailInfoView updateLedgerDetail(
             User user,
-            Ledger ledger,
             LedgerDetail ledgerDetail,
             UpdateLedgerRequest updateLedgerRequest
     ) {
+        long ledgerDetailId = ledgerDetail.getId();
 
-//        LedgerDetail createdLedgerDetail = createLedgerDetail(ledger,
-//                user,
-//                updateLedgerRequest.getStoreInfo(),
-//                ledgerDetail.getFundType(),
-//                updateLedgerRequest.getAmount(),
-//                ledger.getTotalBalance(),
-//                updateLedgerRequest.getDescription(),
-//                updateLedgerRequest.getPaymentDate()
-//        );
+        Ledger ledger = ledgerDetail.getLedger();
 
-        long ledgerDetailId = ledgerDetail.getId();
+        int newAmount = AmountCalculatorByFundType.calculate(
+                ledgerDetail.getFundType(),
+                ledgerDetail.getAmount() - updateLedgerRequest.getAmount()
+        );
+
+        ledger.updateTotalBalance(-newAmount);
+
+        ledgerDetail.updateLedgerDetailInfo(
+                updateLedgerRequest.getStoreInfo(),
+                updateLedgerRequest.getAmount(),
+                updateLedgerRequest.getDescription(),
+                updateLedgerRequest.getPaymentDate()
+        );
+
+        AgencyUser agencyUser = getAgencyUser(user, ledgerDetail);
+
+        validateStaffUserRole(agencyUser.getAgencyUserRole());
+
+        updateBalance(ledger);
 
         // 2. 장부 상세 내역 조회
         List<LedgerReceipt> ledgerReceipts = ledgerReceiptReader.getLedgerReceipts(ledgerDetailId);
@@ -151,13 +148,9 @@ public void removeLedgerDetail(
 
         int newAmount = AmountCalculatorByFundType.calculate(ledgerDetail.getFundType(), ledgerDetail.getAmount());
 
-        ledgerDetailRepository.bulkUpdateLedgerDetailBalance(
-                ledger,
-                ledgerDetail.getPaymentDate(),
-                -newAmount
-        );
-
         ledger.updateTotalBalance(-newAmount);
+
+        updateBalance(ledger);
     }
 
     private void validateStaffUserRole(AgencyUserRole userRole) {
@@ -183,4 +176,24 @@ private User getUser(Long userId) {
                 .findById(userId)
                 .orElseThrow(() -> new NotFoundException(ErrorCode.USER_NOT_FOUND));
     }
+
+    private void updateBalance(Ledger ledger) {
+        List<LedgerDetail> ledgerDetails = ledgerDetailRepository.findAllByLedger(ledger);
+
+        ledgerDetails.sort(Comparator.comparing(LedgerDetail::getPaymentDate));
+
+        int previousBalance = 0;
+
+        for (int i = 0; i < ledgerDetails.size(); i++) {
+            LedgerDetail detail = ledgerDetails.get(i);
+
+            if (detail.getFundType() == INCOME) {
+                previousBalance += detail.getAmount();
+            } else {
+                previousBalance -= detail.getAmount();
+            }
+
+            detail.updateBalance(previousBalance);
+        }
+    }
 }
diff --git a/src/main/java/com/moneymong/domain/ledger/service/manager/LedgerService.java b/src/main/java/com/moneymong/domain/ledger/service/manager/LedgerService.java
@@ -17,7 +17,6 @@
 import com.moneymong.global.exception.custom.InvalidAccessException;
 import com.moneymong.global.exception.custom.NotFoundException;
 import com.moneymong.global.exception.enums.ErrorCode;
-import com.moneymong.utils.AmountCalculatorByFundType;
 import java.util.List;
 import lombok.RequiredArgsConstructor;
 import lombok.extern.slf4j.Slf4j;
@@ -100,36 +99,12 @@ public LedgerDetailInfoView updateLedger(
             final Long ledgerDetailId,
             final UpdateLedgerRequest updateLedgerRequest
     ) {
-        // === 유저 ===
         User user = getUser(userId);
 
-        // === 장부 ===
         LedgerDetail ledgerDetail = getLedgerDetail(ledgerDetailId);
 
-        Ledger ledger = ledgerDetail.getLedger();
-
-        // === 소속 ===
-        AgencyUser agencyUser = getAgencyUser(userId, ledger);
-
-        // === 권한 ===
-        validateStaffUserRole(agencyUser.getAgencyUserRole());
-
-        ledgerDetailService.createLedgerDetail(
-                ledger,
-                user,
-                updateLedgerRequest.getStoreInfo(),
-                ledgerDetail.getFundType(),
-                updateLedgerRequest.getAmount(),
-                ledger.getTotalBalance(),
-                updateLedgerRequest.getDescription(),
-                updateLedgerRequest.getPaymentDate()
-        );
-
-        ledgerDetailService.removeLedgerDetail(userId, ledgerDetailId);
-
         return ledgerDetailService.updateLedgerDetail(
                 user,
-                ledger,
                 ledgerDetail,
                 updateLedgerRequest
         );

diff --git a/src/main/java/com/moneymong/domain/user/api/request/UserDeleteRequest.java b/src/main/java/com/moneymong/domain/user/api/request/UserDeleteRequest.java
@@ -1,13 +1,14 @@
 package com.moneymong.domain.user.api.request;
 
 import jakarta.validation.constraints.NotBlank;
+import lombok.AllArgsConstructor;
 import lombok.Getter;
+import lombok.NoArgsConstructor;
 
 @Getter
+@AllArgsConstructor
+@NoArgsConstructor
 public class UserDeleteRequest {
     @NotBlank
     private String provider;
-
-    @NotBlank
-    private String token;
 }