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 구현 #22

Open
wants to merge 100 commits into
base: team-12
Choose a base branch
from
Open

[BE] 이슈 전체 조회 API 구현 #22

wants to merge 100 commits into from

Conversation

kyupid
Copy link
Collaborator

@kyupid kyupid commented Jun 15, 2021

안녕하세요. 12팀의 백엔드의 MJ, Kyu 입니다.

진행 상황

  • 이슈 전체 조회 API 구현

고민했던 부분

  • 테스트 코드 기반으로 기능을 구현하는데 신경썼습니다.
  • 다른 레이어에 의존하지 않는 단위테스트를 구현하는데 집중했습니다.
  • 특히 컨트롤러에 대한 단위테스트의 경우 서비스 레이어를 mock으로 구현했기 때문에 컨트롤러 내부의 로직에 대한 단위테스트가 가능합니다.

리뷰를 바라는 부분

  • 검색을 통해 각각의 레이어에 적합한 테스트 방식을 찾아서 도입했습니다. 혹시 리뷰어님께서 보시기에 더 나은 방식으로 단위테스트가 가능하다면 조언 부탁드립니다.

kyupid and others added 30 commits June 9, 2021 20:59
@kyupid
Chore: .gitignore 업데이트
e2b1d06
@kyupid
[#11] Feat: 깃헙 Oauth로 AccessToken 받아오기
09b57c2
@kyupid
[#11] Add: 깃헙 서버에 AccessToken 을 담은 요청
f9a01cd
@kyupid
[#11] Add: 깃헙에 AccessToken을 가지고 요청하고 응답으로 커스텀 객체로 유저정보를 받음
d16d1dd
@kyupid
[#11] Feat: Oauth 통해서 JWT 발급
730ad1e
@kyupid
[#14] Feat: 모든 api에 대한 Interceptor 적용
facf310
@kyupid
[#14] Feat: Interceptor를 통해서 JWT 분석 후 특정 API로 데이터를 가져옴
4f7c1b3
@MJbae
프로젝트 세팅 및 로그인 구현
f716cbb 
Oauth 통한 JWT 발급 그리고 Interceptor 구현
@kyupid
Chore: build.gradle 커밋
c0f4dd7
@kyupid
[#27] Feat: 이슈 전체 조회 API 구현 & 테스트
18bfbd7
@kyupid
Add: 스키마, 데이터 초기화
0fe9b1c
@MJbae
Merge pull request #29 from kyu-kim-kr/BE/feature
e7be315 
[#27] 이슈 전체 조회 API 구현을 위한 테스트
@kyupid
[#27] Add: 예제 데이터 추가
91301f3
@kyupid
[#27] 이슈 전체 조회 API 구현 중 문제
760198c 
- Entity -> DTO
@kyupid
Chore: gradle 관련 파일 업로드
2e9fc29 
- 프로젝트 빌드 관련
@kyupid
Chore: 메인 클래스
c644bc9
@kyupid
Add: 유저 정보 받아서 DTO로 넣기
7552c48
@kyupid
Merge branch 'BE/kyu' into BE/feature
22ebdf6
@kyupid
Add: User 도메인에 기본 생성자 추가
8798b86
@kyupid
Add: Issue 도메인에 List<Label> 추가
e0f7bd1
@kyupid
[#27] Fix: RowMapper 사용 방식 수정
177feb9
@kyupid
[#27] Add: Milestone 도메인, 레포지토리 추가
12fcaee
@kyupid
[#27] Add: IssueDto에 MilestoneTitle 가져오기
ad330fa
@kyupid
[#27] Add: JsonPropery 어노테이션 추가
ac8d2c2
@kyupid
[#27] Add: Label 도메인, 레포지토리 추가
fe62345
chore: 프로젝트 설정
60d36ee
chore: 인터셉터 및 gitignore 설정
fbd6b43
test: 이슈 전체조회 테스트 구현
c103033
test: 테스트 목적의 issue 객체에 대해 클래스 분리
c78e135
test: IssueController에 대한 테스트 구현
08297a4
MJbae and others added 5 commits June 22, 2021 14:59
feat: 코멘트 삭제 기능 구현
975221b
@kyupid
Feat: 담당자/작성자 조회
1a836b8
@kyupid
Merge pull request #87 from kyu-kim-kr/kyu/temp
a68fb63 
Kyu/temp
@kyupid
Merge pull request #88 from kyu-kim-kr/BE/feat-comment
e2ad46d 
Be/feat comment
@kyupid
[#91] Fix/Add: 2021.06.22 회의 후 수정 사항
98b7982 
- [x] 이슈상세조회에 라벨에 font_light 넣기
- [x] 이슈전체조회 assignee에 s하나 붙이기
- [x] 이슈상세조회 assignee 수정
- [x] 이슈전체조회를 했을때  assignees 배열로 반환
- [x] 이슈코멘트카운트 속성 추가
Sonjh1306 added a commit that referenced this pull request Jun 22, 2021
@Sonjh1306
[#22] feat: 커스텀 헤더뷰 구현
2762344
Sonjh1306 added a commit that referenced this pull request Jun 22, 2021
@Sonjh1306
[#22] feat: 이슈 생성 화면 UI 구현
cc7dd0e
sonypark pushed a commit that referenced this pull request Jun 23, 2021
@ChoiGiSung
Merge pull request #22 from ChoiGiSung/image
80272df 
[BE] Image 첨부
MJbae and others added 18 commits June 24, 2021 09:48
fix: issue 조회 시, issueRepository에서 엔티티 가져올 때 deleted = false 조건 추가
9412568
@kyupid
Fix: 이슈 조회 내림차순
108e06a
@kyupid
[#113] Feat: 이미지 업로드 API
1669738
@kyupid
Merge pull request #114 from kyu-kim-kr/kyu/temp-image
59628a0 
[BE] [#113] Feat: 이미지 업로드 API
@kyupid
Merge pull request #122 from kyupid/BE/feature
0e6d406 
[BE] 병합
@kyupid
Fix: 초기화 데이터 라벨 색상 변경
43856a4
@kyupid
Fix/Add: 테스트 삭제, 필요한 테스트 추가
17505c3
@kyupid
Fix bug: 날짜 매핑하지 못하는 문제 해결
2d6027c
@kyupid
Feat: 이모지 상태 전체 조회
afc8cc7
@kyupid
Merge pull request #129 from kyupid/kyu/feature/get-emojis-status
e8abfe5 
Feat: 이모지 상태 전체 조회
@kyupid
Rename: 변수이름 명확히 하기 위해 수정
554de00
@kyupid
[#127] Feat: 이모지 달기 구현
9d68098
@kyupid
Merge pull request #133 from kyupid/kyu/feature/emoji
5a429d0 
Kyu/feature/emoji 완성
@kyupid
[#136] Remove: 필요없는 코드/주석 제거
1c481d5
@kyupid
[#136] Fix: 이미지 업로드 리턴타입 수정
8339e42
@kyupid
[FE 요청사항] Fix: 이슈 상제조회시 유저,라벨,마일스톤에 id도 응답하도록 수정
c5be3dd
@MJbae
Merge pull request #138 from kyupid/kyu/rebase
c367cdd 
[BE] 리팩토링용 PR
@kyupid
Merge pull request #146 from kyupid/BE/feature
ec2fc48 
Be/feature
janeljs pushed a commit that referenced this pull request Jul 13, 2021
@eNoLJ
feat: (#22) 헤더 검증 방식 변경
a3f0b2d
janeljs pushed a commit that referenced this pull request Jul 13, 2021
@eNoLJ
feat: (#22) 로그인 시 헤더 검증 방법 변경
80519c3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wheejuni wheejuni wheejuni requested changes

Assignees

@wheejuni wheejuni

@kyupid kyupid

Labels
review-BE
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@kyupid @wheejuni @MJbae