Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[BE] 이슈 필터링 API 구현 #58

Open
MuseopKim opened this issue Jun 18, 2020 · 0 comments
Open

[BE] 이슈 필터링 API 구현 #58

MuseopKim opened this issue Jun 18, 2020 · 0 comments
Assignees
@hanurii @MuseopKim
Labels
BE 백엔드 이슈

Comments

@MuseopKim
Copy link
Member

이슈 필터링 기능

  • open / closed
  • author
  • label
  • milestones
  • assignee
@MuseopKim MuseopKim added the BE 백엔드 이슈 label Jun 18, 2020
@hanurii hanurii modified the milestone: FE WEEK 3 Jun 21, 2020
MuseopKim added a commit that referenced this issue Jun 25, 2020
@MuseopKim
[BE/#58] Feat : 이슈 필터링 기능 컨트롤러 로직 추가
f191697 
-> issueService.findIssuesByFilterParameters()
MuseopKim added a commit that referenced this issue Jun 25, 2020
@MuseopKim
[BE/#58] Feat : 필터링 파라미터 클래스 추가
c7b9389 
- default 값 없이 클래스로 처리
- FilterParameters {}
- open
- author
- label
- milestones
- assignee
MuseopKim added a commit that referenced this issue Jun 25, 2020
@MuseopKim
[BE/#58] Feat : 필터 기능 DAO 쿼리 추가
57c3204 
- 클래스로 처리하여 null 값을 활용
- COALESCE 함수 활용
MuseopKim added a commit that referenced this issue Jun 25, 2020
@MuseopKim
[BE/#58] Feat : 이슈 필터링 기능 컨트롤러 로직 추가
16d0962 
-> issueService.findIssuesByFilterParameters()
MuseopKim added a commit that referenced this issue Jun 25, 2020
@MuseopKim
[BE/#58] Feat : 이슈 필터링 서비스 로직 구현
06fe494 
- M:N 이므로 두 번의 관계 테이블 조인이 필요, 로직을 두 가지로 분할 (Labels와 먼저 조인한 이후 필터 된 id 값으로 Assignee와 한차례 더 조인)
- 두 번째 조인 로직에서 필터링 된 Id 목록이 하나도 없을 경우 비어 있는 이슈 리스트를 리턴 하도록 설계
- 일부 하드 코딩 된 부분 (특히 매직 넘버) 추후 리팩토링
MuseopKim added a commit that referenced this issue Jun 25, 2020
@MuseopKim
[BE/#58] Refactor : DTO 클래스 패키지 이동 및 이름 변경
aa3ac6c 
- Modify--  ==>  Update--
- dto - request, response로  패키지 분리
MuseopKim added a commit that referenced this issue Jun 25, 2020
@MuseopKim
[BE/#58] Refactor : 응답 DTO, VO 프론트단의 요청에 따라 필드 값 변경
0662b73 
- id, title, description으로 이름 간소화
- static create() -> static of() 변경
MuseopKim added a commit that referenced this issue Jun 25, 2020
@MuseopKim
[BE/#58] Refactor : DTO에 따라 마일스톤의 progress 값을 갖도록 분기처리 하는 로직 삭제
7e970d0 
- 이슈 목록의 각 이슈에 할당된 마일스톤에도 progress가 출력 되도록 수정
MuseopKim added a commit that referenced this issue Jun 25, 2020
@MuseopKim
[BE/#58] Refactor : Label 테이블 컬럼 추가에 따른 코드 및 로직 변경
4fffaa9 
- hexCode -> backgroundColor, color 세분화
- 이에 따라 각 레이블 관련 코드 수정 (생성자, DAO Read 로직 위주)
MuseopKim added a commit that referenced this issue Jun 25, 2020
@MuseopKim
[BE/#58] Refactor : 이슈 상세 정보 JSON 변화에 따른 코멘트 관련 DTO 및 VO 수정
eda2cf7 
- commentInfo 생성
- countOfComment를 comment 리스트와 관련 객체에 함께 포함
MuseopKim added a commit that referenced this issue Jun 25, 2020
@MuseopKim
[BE/#58] Refactor : 이슈 상세 정보 DTO 생성 로직 수정
b69b2a1 
- JSON 변동에 따라 comment 리스트를 인자로 전달하는 로직에서 commentInfo 클래스를 인자로 추가하는 로직으로 변경
MuseopKim added a commit that referenced this issue Jun 25, 2020
@MuseopKim
[BE/#58] Refactor : 이슈 상태 일괄 수정 기능 추가에 따른 로직 변경
720aea1 
- 여러개의 이슈에 대해 open / closed 상태를 일괄 변경 할 수 있도록 변경
- 기능 관련 수정은 일단락, 이후 리팩토링에선 컨트롤러 및 불필요한 getter / setter / 생성자 정리 필요
MuseopKim added a commit that referenced this issue Jun 25, 2020
@MuseopKim
[BE/#58] Refactor : JSON 구조 변경에 따른 DTO, VO 클래스 수정
c109865 
- commenter 객체안에 있던 필드 값을 상위 객체로 가져와서 comment 정보와 함께 사용 (userId, name, avatarUrl)
- 필드 값 변화에 따른 CommentDetails 생성자 수정
MuseopKim added a commit that referenced this issue Jun 25, 2020
@MuseopKim
[BE/#58] Fix : labelInfo JSON 변경에 따른 DTO, VO 수정
889072c 
- LabelSummary -> LabelDetails
- description 필드 추가
- JsonInclude annotation을 활용하여 분기 처리
MuseopKim added a commit that referenced this issue Jun 25, 2020
@MuseopKim
[BE/#58] Fix : 레이블 DTO, VO 변경사항 적용 완료
f3d1e4d 
- LabelInformation의 경우 description을 인자로 추가하여 생성 하도록 수정
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hanurii hanurii

@MuseopKim MuseopKim

Labels
BE 백엔드 이슈
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@hanurii @MuseopKim