-
Notifications
You must be signed in to change notification settings - Fork 1
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.
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
[BE] 이슈 목록 API 구현 #17
Comments
hanurii
added a commit
that referenced
this issue
Jun 11, 2020
hanurii
added a commit
that referenced
this issue
Jun 11, 2020
- 원하고자 하는 응답 형태로는 만들었다 - 하지만 오늘 랙돌님과 피드백한 JSON 응답 형태로는 만들지 않았기 때문에 그 작업을 진행할 예정
hanurii
added a commit
that referenced
this issue
Jun 11, 2020
hanurii
added a commit
that referenced
this issue
Jun 11, 2020
- 랙돌님과 상의한 부분까지 수정 완료(DTO 변수명 변경, milestone, author 객체형태로 응답) - 프론트 분들이 필터에 들어가는 전체 데이터까지 한번에 받길 원하면 그 기능을 구현할 예정
hanurii
added a commit
that referenced
this issue
Jun 11, 2020
hanurii
added a commit
that referenced
this issue
Jun 11, 2020
hanurii
added a commit
that referenced
this issue
Jun 11, 2020
MuseopKim
added a commit
that referenced
this issue
Jun 12, 2020
user 테이블에 이미지 url 컬럼 추가 - avatar_url 테이블명 변경 - issue_has_label RDS 사용으로 전환
MuseopKim
added a commit
that referenced
this issue
Jun 12, 2020
DB의 테이블과 같은 구성을 가진 도메인 클래스 생성 - User - Milestone - Issue
MuseopKim
added a commit
that referenced
this issue
Jun 12, 2020
DB의 테이블과 같은 구성을 가진 도메인 클래스 생성 - Comment - Assignee (다대다 관계 테이블, 불 필요시 추후 삭제할 수 있다.)
MuseopKim
added a commit
that referenced
this issue
Jun 12, 2020
ApiResponse - 상태와(혹은 추후 상태코드로 변경) 응답 객체를 반환하는 응답 클래스 추가
MuseopKim
added a commit
that referenced
this issue
Jun 12, 2020
DB 테이블과 같은 구성을 가진 도메인 클래스 생성 - Label
MuseopKim
added a commit
that referenced
this issue
Jun 12, 2020
- UserSummary : Author, Assignees 등 유저 정보를 다루는 VO 역할 - MilestoneSummary : 마일스톤에 대한 간략 정보를 다루는 VO - LabelDetails : 레이블에 대한 정보를 담고 있는 VO (도메인과 중복되어 역할에 대한 고민이 더 필요)
MuseopKim
added a commit
that referenced
this issue
Jun 12, 2020
- LabelInformation : 등록된 레이블의 갯수, 레이블 상세 정보 리스트를 포함 - MilestoneInformation : 등록된 마일스톤의 갯수, 각 마일스톤에 대한 상세 정보 리스트 포함
MuseopKim
added a commit
that referenced
this issue
Jun 12, 2020
- issueId : 이슈의 아이디 값 - issueTitle : 이슈 제목 - milestone : 소속 마일스톤 정보 - attachedLabels : 해당 이슈에 할당 된 레이블 정보 - author : 해당 이슈의 작성자 - allocatedAssignees : 해당 이슈에 할당 된 담당자 정보 - createdAt : 이슈 생성 날짜 및 시간 정보 - opened : open되어 있는 이슈인지, closed 되어 있는 이슈인지에 관한 정보와 연관
MuseopKim
added a commit
that referenced
this issue
Jun 12, 2020
- issues : 각 이슈에 관한 정보 배열 - labelInfo : 등록되어 있는 레이블 갯수와 각 레이블에 대한 상세 정보 필드 - milestoneInfo : 등록 되어 있는 마일스톤의 갯수와 각 마일스톤에 대한 상세 정보 필드 - assigneeInfo : assignee로 지정할 수 있는 유저 리스트
MuseopKim
added a commit
that referenced
this issue
Jun 12, 2020
- ResponseEntity를 활용하여 ApiResponse에 담겨 있는 바디 값과 상태코드를 반환 하도록 설계
MuseopKim
added a commit
that referenced
this issue
Jun 12, 2020
MuseopKim
added a commit
that referenced
this issue
Jun 12, 2020
- Milestone : 자료형 List -> Set 변경 - Comment : 필드 값 오타 수정 issuId -> issueId - Assignee : getter 메서드 라인 정리
MuseopKim
added a commit
that referenced
this issue
Jun 12, 2020
- Issue : 기본 Builder 생성자 추가 - User : githubId 자료형 변경 (Integer -> Long) - LabelSummary : 클래스명 변경 (LabelDetails -> LabelSummary) - LabelInformation : 필드 값 labels 자료형 변경 (List -> Set)
MuseopKim
added a commit
that referenced
this issue
Jun 12, 2020
- 모든 레이블 목록을 반환하는 쿼리 추가 - 해당 이슈에 할당된 레이블을 반환하는 쿼리 추가 - SQL을 최대한 작은 단위로 활용
MuseopKim
added a commit
that referenced
this issue
Jun 12, 2020
MuseopKim
added a commit
that referenced
this issue
Jun 12, 2020
- Issue 도메인을 IssueDeatils로 만드는 로직을 mapToIssueDeatils 메서드 분리
hanurii
added a commit
that referenced
this issue
Jun 18, 2020
- DTO, VO, ENTITIY를 공부한 후 전체적인 로직을 그리며 구현 - 아직 변수명이나 타입들을 수정해야 할 부분들이 있음
hanurii
added a commit
that referenced
this issue
Jun 18, 2020
- JSON 응답 객체에 SET 타입으로 넘기니까 id 값이 순서대로 들어가지 않고 뒤죽박죽 들어감 - 가독성을 헤치는 것 같아서 변경
hanurii
added a commit
that referenced
this issue
Jun 19, 2020
hanurii
added a commit
that referenced
this issue
Jun 19, 2020
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The text was updated successfully, but these errors were encountered: