CLAP-153 작업 히스토리 조회 by parkjaehak · Pull Request #158 · TaskFlow-CLAP/TaskFlow-Server

parkjaehak · 2025-01-29T01:27:33Z

📄 요약(Summary)

작업 히스토리 조회

✍🏼 상세(More)

작업 상세 내역에 존재하는 작업 히스토리를 조회합니다.

PR Desciption

변경 사항 설명

comment와 연관관계가 존재하는 attachment 조회
querydsl을 사용해 댓글, 댓글첨부파일, 작업상태변경, 담당자변경에 대한 모든 taskhistory 조회
응답 매핑시 taskhistorystatus에 따라 그에 맞는 응답 객체에 매핑하여 전달

Requirements for Reviewer

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요
ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

PR Log

PR 작업하면서 고민했던 내용, 해결한 내용, 고민 중인 내용 등

새롭게 배운 것

고민 중인 사항

응답 객체가 아래와 같은 형태로 케이스마다 갈 것 같은데 details에 해당되는 부분만 응답으로 보내고 해당되지 않는 부분은 null이 아닌 아무것도 없는 형태로 보내는게 가능한지 모르겠네요. 구현상 어쩔수 없이 해당되지 않는 부분은 null이 반환됩니다.
아래 사진에서 modifiedField와 newValue는 사실상 필요가 없지 않나 생각이 듭니다. TaskHistoryType로 modifiedField를 대신할 수 있고 modifiedMember로 newValue를 대체할 수 있다고 생각이 들어서요. 이부분에 대한 의견을 듣고 싶습니다.
MapStruct를 이용해 Entity를 domain으로 매핑하는 과정에서 필요로 하지 않는 부분까지 로드되는 문제가 있는 것 같습니다.
모든 PersistenceMapper에서 domain을 매핑시 연관관계에 있는 객체를 호출하는데 이때 lazy로딩이 걸려 쿼리가 두번 나가는 것 같은데 필요한 부분만 매핑하고 실제 필요한 연관관계의 객체는 수동으로 매핑해주는 과정이 필요하지 않나라고 생각이 듭니다.

첨부 자료

{
    "histories": [
        {
            "historyId": 1,
            "date": "2025-01-12",
            "time": "14:18:00",
            "taskHistoryType": "PROCESSOR_ASSIGNED",
            "details": {
                "processorChanged": {
                    "previousProcessor": null,
                    "currentProcessor": "bac.park"
                },
                "taskStatusSwitched": null,
                "commentDetails": null,
                "commentFileDetails": null
            }
        },
        {
            "historyId": 2,
            "date": "2025-01-13",
            "time": "14:18:00",
            "taskHistoryType": "STATUS_SWITCHED",
            "details": {
                "processorChanged": null,
                "taskStatusSwitched": {
                    "taskStatus": "진행 중"
                },
                "commentDetails": null,
                "commentFileDetails": null
            }
        },
        {
            "historyId": 3,
            "date": "2025-01-13",
            "time": "14:19:00",
            "taskHistoryType": "COMMENT",
            "details": {
                "processorChanged": null,
                "taskStatusSwitched": null,
                "commentDetails": {
                    "name": "ldk.park",
                    "profileImageUrl": "https://image.png",
                    "isModified": false,
                    "comment": "일반 댓글"
                },
                "commentFileDetails": null
            }
        },
        {
            "historyId": 4,
            "date": "2025-01-13",
            "time": "14:20:00",
            "taskHistoryType": "COMMENT_FILE",
            "details": {
                "processorChanged": null,
                "taskStatusSwitched": null,
                "commentDetails": null,
                "commentFileDetails": {
                    "name": "atom.park",
                    "profileImageUrl": "https://image.png",
                    "isModified": false,
                    "fileName": "수정된 파일 이름",
                    "url": "https://수정경로8.png",
                    "size": "17MB"
                }
            }
        },
        {
            "historyId": 5,
            "date": "2025-01-14",
            "time": "09:00:00",
            "taskHistoryType": "PROCESSOR_CHANGED",
            "details": {
                "processorChanged": {
                    "previousProcessor": null,
                    "currentProcessor": "ldk.park"
                },
                "taskStatusSwitched": null,
                "commentDetails": null,
                "commentFileDetails": null
            }
        }
    ]
}

Requirements for Reviewer

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

✅ 체크리스트(Checklist)

PR 양식에 맞게 작성했습니다
모든 테스트가 통과했습니다
프로그램이 정상적으로 작동합니다
적절한 PR 라벨을 설정했습니다
불필요한 코드를 제거했습니다

🚪 이슈 번호(Issue numbers)

Closes #130

joowojr

2.modified_field는 history type으로 대체할 수 있다는 의견에는 동감합니다. 하지만 new value는 상태 전환의 경우도 있기 때문에 필요하다고 생각됩니다!

joowojr

응답 json 구조는 아래와 같이 더 유연하게 나타낼 수 있을 것 같습니다!
comment일 경우에는 comment, commentfile 둘 중 하나가 null로 들어가게 되겠네요

아니면 프론트와 협의하에 comment와 commentfile을 합쳐서 파일첨부가 아닐 경우에는 filesize 필드를 null로 처리해도될것같아요.

{
   "historyId":1,
   "date":"2025-01-12",
   "time":"14:18:00",
   "taskHistoryType":"PROCESSOR_ASSIGNED", // UI 조건부 렌더링 및 json 파싱을 위한 값
   "details":{
      "task":{
         "modifiedValue":"processor", // 있어도 되고 없어도 될듯
         "newValue":"siena.it"
      },
      "comment":{
         "content":"댓글",
         "memberId":2,
         "profileImg": "프로필 이미지 주소"
      },
      "commentFile":{
         "content":"파일 주소",
         "fileSize": "16MB",
         "memberId":2,
         "profileImg": "프로필 이미지 주소"
      }
   }
}

joowojr

네 저도 이 부분을 어제 알게 되어서 lazy 로딩 관계의 경우에는 해당 매핑을 수동으로 처리해야할 것 같습니다..! 이 부분은 오늘 회의에서 한번 더 얘기하면 좋을것 같아요

parkjaehak · 2025-01-29T02:47:18Z

2.modified_field는 history type으로 대체할 수 있다는 의견에는 동감합니다. 하지만 new value는 상태 전환의 경우도 있기 때문에 필요하다고 생각됩니다!

이 부분은 제가 TaskModificationInfo의 task를 기존 taskid에서 task로 변경하여 domain 매핑시에 taskStatus를 가져올 수 있도록 설정하였고 task의 상태가 바뀔 경우 new value 대신 taskStatus의 description을 가져와 바뀐 상태를 표시해주면 되지 않을까 생각했습니다.
또한 TaskModificationInfo에 modifiedMemberId도 참조하고 있기 때문에 이를 통해 processor의 닉네임을 가져올 수 있기 때문에 dto 매핑시에 직접 가져오도록 하면 상태나 담당자가 변경될때 newValue에 값을 쓰지 않아도 된다고 판단했는데 이 부분 의견을 듣고 싶습니다.

parkjaehak · 2025-01-29T03:02:36Z

응답 json 구조는 아래와 같이 더 유연하게 나타낼 수 있을 것 같습니다!
comment일 경우에는 comment, commentfile 둘 중 하나가 null로 들어가게 되겠네요

아니면 프론트와 협의하에 comment와 commentfile을 합쳐서 파일첨부가 아닐 경우에는 filesize 필드를 null로 처리해도될것같아요.
{
   "historyId":1,
   "date":"2025-01-12",
   "time":"14:18:00",
   "taskHistoryType":"PROCESSOR_ASSIGNED", // UI 조건부 렌더링 및 json 파싱을 위한 값
   "details":{
      "task":{
         "modifiedValue":"processor", // 있어도 되고 없어도 될듯
         "newValue":"siena.it"
      },
      "comment":{
         "content":"댓글",
         "memberId":2,
         "profileImg": "프로필 이미지 주소"
      },
      "commentFile":{
         "content":"파일 주소",
         "fileSize": "16MB",
         "memberId":2,
         "profileImg": "프로필 이미지 주소"
      }
   }
}

task 에 대한 부분을 하나로 합치도록 해보겠습니다.

parkjaehak · 2025-01-29T06:12:55Z

현재 응답 구조는 아래와 같은 형태로 수정하였습니다.

{
    "histories": [
        {
            "historyId": 1,
            "date": "2025-01-12",
            "time": "14:18:00",
            "taskHistoryType": "PROCESSOR_ASSIGNED",
            "details": {
                "taskDetails": {
                    "newValue": "bac.park"
                },
                "commentDetails": null,
                "commentFileDetails": null
            }
        },
        {
            "historyId": 2,
            "date": "2025-01-13",
            "time": "14:18:00",
            "taskHistoryType": "STATUS_SWITCHED",
            "details": {
                "taskDetails": {
                    "newValue": "진행 중"
                },
                "commentDetails": null,
                "commentFileDetails": null
            }
        },
        {
            "historyId": 3,
            "date": "2025-01-13",
            "time": "14:19:00",
            "taskHistoryType": "COMMENT",
            "details": {
                "taskDetails": null,
                "commentDetails": {
                    "name": "ldk.park",
                    "profileImageUrl": "https://image.png",
                    "isModified": false,
                    "comment": "일반 댓글"
                },
                "commentFileDetails": null
            }
        },
        {
            "historyId": 4,
            "date": "2025-01-13",
            "time": "14:20:00",
            "taskHistoryType": "COMMENT_FILE",
            "details": {
                "taskDetails": null,
                "commentDetails": null,
                "commentFileDetails": {
                    "name": "atom.park",
                    "profileImageUrl": "https://image.png",
                    "isModified": false,
                    "fileName": "수정된 파일 이름",
                    "url": "https://수정경로8.png",
                    "size": "17MB"
                }
            }
        },
        {
            "historyId": 5,
            "date": "2025-01-14",
            "time": "09:00:00",
            "taskHistoryType": "PROCESSOR_CHANGED",
            "details": {
                "taskDetails": {
                    "newValue": "ldk.park"
                },
                "commentDetails": null,
                "commentFileDetails": null
            }
        }
    ]
}

joowojr · 2025-01-29T08:51:10Z

현재 응답 구조는 아래와 같은 형태로 수정하였습니다.

{
    "histories": [
        {
            "historyId": 1,
            "date": "2025-01-12",
            "time": "14:18:00",
            "taskHistoryType": "PROCESSOR_ASSIGNED",
            "details": {
                "taskDetails": {
                    "newValue": "bac.park"
                },
                "commentDetails": null,
                "commentFileDetails": null
            }
        },
        {
            "historyId": 2,
            "date": "2025-01-13",
            "time": "14:18:00",
            "taskHistoryType": "STATUS_SWITCHED",
            "details": {
                "taskDetails": {
                    "newValue": "진행 중"
                },
                "commentDetails": null,
                "commentFileDetails": null
            }
        },
        {
            "historyId": 3,
            "date": "2025-01-13",
            "time": "14:19:00",
            "taskHistoryType": "COMMENT",
            "details": {
                "taskDetails": null,
                "commentDetails": {
                    "name": "ldk.park",
                    "profileImageUrl": "https://image.png",
                    "isModified": false,
                    "comment": "일반 댓글"
                },
                "commentFileDetails": null
            }
        },
        {
            "historyId": 4,
            "date": "2025-01-13",
            "time": "14:20:00",
            "taskHistoryType": "COMMENT_FILE",
            "details": {
                "taskDetails": null,
                "commentDetails": null,
                "commentFileDetails": {
                    "name": "atom.park",
                    "profileImageUrl": "https://image.png",
                    "isModified": false,
                    "fileName": "수정된 파일 이름",
                    "url": "https://수정경로8.png",
                    "size": "17MB"
                }
            }
        },
        {
            "historyId": 5,
            "date": "2025-01-14",
            "time": "09:00:00",
            "taskHistoryType": "PROCESSOR_CHANGED",
            "details": {
                "taskDetails": {
                    "newValue": "ldk.park"
                },
                "commentDetails": null,
                "commentFileDetails": null
            }
        }
    ]
}

수고하셨습니다! 회원 정보에 name만 nickname으로 수정하면 좋을것같아요:)

parkjaehak added ✨ feature 구현·개선 사항에 관련된 내용입니다 HIGH 우선순위 상 labels Jan 29, 2025

parkjaehak self-assigned this Jan 29, 2025

joowojr reviewed Jan 29, 2025

View reviewed changes

parkjaehak force-pushed the CLAP-153 branch 2 times, most recently from 0c3e1b7 to 98f794e Compare January 29, 2025 06:05

parkjaehak added 7 commits January 29, 2025 22:52

CLAP-153 Refactor: application task 패키지 service 계층으로 이동

2c7995c

CLAP-153 Feat: 작업 히스토리 조회

9850852

CLAP-153 Fix: 불필요한 주석 및 로직 제거

578440e

CLAP-153 Fix: 작업 조회 스웨거 태그 설정

90547f4

CLAP-153 Fix: 이전 처리자 이름 매핑 삭제

3d3019d

CLAP-153 Fix: 작업 히스토리 응답 구조 변경

a118dc9

CLAP-153 Fix: 댓글 작성자 nickName으로 변수명 변경

a3c0bd5

parkjaehak force-pushed the CLAP-153 branch from d175911 to a3c0bd5 Compare January 29, 2025 13:52

parkjaehak merged commit 250507d into develop Jan 29, 2025
1 check passed

parkjaehak deleted the CLAP-153 branch February 3, 2025 05:56

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

CLAP-153 작업 히스토리 조회#158

CLAP-153 작업 히스토리 조회#158
parkjaehak merged 7 commits intodevelopfrom
CLAP-153

parkjaehak commented Jan 29, 2025 •

edited

Loading

Uh oh!

joowojr left a comment •

edited

Loading

Uh oh!

joowojr left a comment •

edited

Loading

Uh oh!

joowojr left a comment

Uh oh!

parkjaehak commented Jan 29, 2025 •

edited

Loading

Uh oh!

parkjaehak commented Jan 29, 2025

Uh oh!

parkjaehak commented Jan 29, 2025

Uh oh!

joowojr commented Jan 29, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

parkjaehak commented Jan 29, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

📄 요약(Summary)

PR Desciption

Requirements for Reviewer

PR Log

새롭게 배운 것

고민 중인 사항

첨부 자료

Requirements for Reviewer

✅ 체크리스트(Checklist)

🚪 이슈 번호(Issue numbers)

Uh oh!

joowojr left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

joowojr left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

joowojr left a comment

Choose a reason for hiding this comment

Uh oh!

parkjaehak commented Jan 29, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

parkjaehak commented Jan 29, 2025

Uh oh!

parkjaehak commented Jan 29, 2025

Uh oh!

joowojr commented Jan 29, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

parkjaehak commented Jan 29, 2025 •

edited

Loading

joowojr left a comment •

edited

Loading

joowojr left a comment •

edited

Loading

parkjaehak commented Jan 29, 2025 •

edited

Loading