• project table에 start_date column 추가
• project table에 end_date column 추가
• delete @AllArgsConstructor
• add Constructor and @builder 어노테이션

project?location=ALL&position=ALL 쿼리스트링에 ALL 을 넣으면
사용자 입장에서 전체 메뉴를 선택할 수 도 있고, 분기 처리할 때 경우의 수도 줄어들거 같습니다~

프로젝트 개설 페이지

• 현재 모집 인원이 0 명일 경우에도 프로젝트가 정상적으로 개설되는 버그가 있습니다.

• 개발 직군에 기타라는 항목이 있기 때문에, 프로젝트는 최소 1명 이상일 때, 개설 되어야 합니다. 해당 문제점을 해결하고 아래에 댓글을 달아주세요!

• 각 페이지별 데이터처리를 위한 DTO 생성

• 기존 단일 도메인으로 처리 할 경우 순환참조 문제가 있음

  • 관련 문제 참고자료

CORS가 뜨네용

• 프로젝트가 가지고 있는 기술 태그를 기반으로 한 검색 구현
• 태그 검색 API 작성

제안

주로 리스트에 쓰이는 데이터(ex.Project List와 Comment List 의 데이터) 포맷이 _embaded.xxxDTOList.Array 로 날라오고 단일 데이터는 단일객체로 날라오는데, 데이터 포맷을 아래와 같은 형식으로 바꾸는 것을 제안

List 데이터 포맷

• comment list를 예로 들어,

기존 포맷

{
    "_embedded": {
        "commentDTOList": [
            {
                "userName": "testUser_25",
                "projectTitle": "이러 이러한 Side Project 의 함께할 사람들을 찾고 있습니다. 1",
                "content": "테스트 댓글 162",
                "createdDate": "2019-08-12T21:20:55",
                "modifiedDate": null,
                "_links": {
                    "self": {
                        "href": "https://donghun-dev.kro.kr:8084/api/comment/162"
                    }
                }
            },
            {
                "userName": "testUser_22",
                "projectTitle": "이러 이러한 Side Project 의 함께할 사람들을 찾고 있습니다. 1",
                "content": "테스트 댓글 236",
                "createdDate": "2019-08-12T21:20:56",
                "modifiedDate": null,
                "_links": {
                    "self": {
                        "href": "https://donghun-dev.kro.kr:8084/api/comment/236"
                    }
                }
            }
        ]
    },
    "_links": {
        "self": {
            "href": "https://donghun-dev.kro.kr:8084/api/project/1/comments"
        }
    }
}

변경 포맷

{
        "datas": [
            {
                "userName": "testUser_25",
                "projectTitle": "이러 이러한 Side Project 의 함께할 사람들을 찾고 있습니다. 1",
                "content": "테스트 댓글 162",
                "createdDate": "2019-08-12T21:20:55",
                "modifiedDate": null,
                "_links": {
                    "self": {
                        "href": "https://donghun-dev.kro.kr:8084/api/comment/162"
                    }
                }
            },
            {
                "userName": "testUser_22",
                "projectTitle": "이러 이러한 Side Project 의 함께할 사람들을 찾고 있습니다. 1",
                "content": "테스트 댓글 236",
                "createdDate": "2019-08-12T21:20:56",
                "modifiedDate": null,
                "_links": {
                    "self": {
                        "href": "https://donghun-dev.kro.kr:8084/api/comment/236"
                    }
                }
            }
        ]
}

• _embadded 프로퍼티와 최상위_links프로퍼티를 지우고, 배열 부분 commentDTOList -> datas로 프로퍼트 이름을 변경하는 것 제안
• 관례상 리스트 데이터에서 최상위 프로퍼티 이름을datas라는 프로퍼티로 보내거나 아예 배열만 보냄

단일 객체 포맷

• proeject detail을 예로 들어

현재 포맷

{
    "title": "이러 이러한 Side Project 의 함께할 사람들을 찾고 있습니다. 1",
    "leader": "testUser_34",
    "content": "1. 동해물과 백두산이 마르고 닳도록 \n하느님이 보우하사 우리나라 만세\n무궁화 삼천리 화려 강산\n대한 사람 대한으로 길이 보전하세\n\n2. 남산 위에 저 소나무 철갑을 두른 듯 \n바람 서리 불변함은 우리 기상일세\n무궁화 삼천리 화려 강산 \n대한 사람 대한으로 길이 보전하세\n\n3. 가을 하늘 공활한데 높고 구름 없이 \n밝은 달은 우리 가슴 일편단심일세\n무궁화 삼천리 화려 강산\n대한 사람 대한으로 길이 보전하세\n\n4. 이 기상과 이 맘으로 충성을 다하여 \n괴로우나 즐거우나 나라 사랑하세\n무궁화 삼천리 화려 강산\n대한 사람 대한으로 길이 보전하세",
    "status": "종료",
    "location": "부산",
    "createdDate": "2019-08-12T21:20:53",
    "deadline": "2019-08-12T21:20:53",
    "startDate": "2019-08-12T21:20:53",
    "endDate": "2019-08-12T21:20:53",
    "modifiedDate": null,
    "developerRecruits": 2,
    "designerRecruits": 0,
    "plannerRecruits": 0,
    "marketerRecruits": 0,
    "etcRecruits": 2,
    "currentDeveloper": 0,
    "currentDesigner": 0,
    "currentPlanner": 1,
    "currentMarketer": 0,
    "currentEtc": 1,
    "summary": "이러한 프로젝트에 참여할 인원을 모집합니다.",
    "_links": {
        "self": {
            "href": "https://donghun-dev.kro.kr:8084/api/project/1"
        },
        "Leader": {
            "href": "https://donghun-dev.kro.kr:8084/api/projects/1/leader"
        },
        "Comments": {
            "href": "https://donghun-dev.kro.kr:8084/api/project/1/comments"
        }
    }
}

• 딱좋음ㅋ👍🏻, 이대로 유지해도 좋음

• User 도메인의 password 컬럼은 객체를 반환할 때, 노출되어서는 안되기 때문에 조치가 필요.

• 도메인의 관계성 매핑을 위한 필드 쪽에서 멀티 벨류 필드를 Set 컬렉션을 사용할 경우, 객체들을 삽입할 때, 그 객체의 내부 필드 객체가 중복 될 경우, 객체 자체의 중복으로 인지하여 hashCode() 함수를 무한 참조하는 StackOverflow 에러가 발생하므로 조치가 필요.

  • 관련 내용

    "ID-based equality" means just @EqualsAndHashCode(of="id"). It just says, two objects are equals if they have the same ID. This can be a problem with newly created objects (having no ID yet), etc. It avoids problems with objects changing their state (as the ID never changes, at least for persisted objects) and so changing their hashCode (such a change corrupts any HashMap/Set).

• 기존 listResource에서 pageResource로 변경
• hateoas 기반의 infinity scroll 구현을 위함

유저 프로필을 get 할때는 자기소개 부분이 summary라는 이름으로 데이터가 날라오는데,
POST 할때는 description 으로 보내게 되어 있으면 정보수정 로직짤때 복잡해져서 둘중 하나로 통일했으면 좋겠다

24개씩 가져오는게 괜찮을거 같습니다~

• status column의 명세 변경 ( 기존 varchar(30) -> varchar(10) )
• UserProject의 status column이 명세와 불일치함 ( boolean -> varchar(10) )
• Project의 status column 변경 필요 ( varchar(30) -> varchar(10) )
• • UserProject class 추가 생성자 필요

• MatchingApplication.class의 static SecurityConfiguration 파일 분리
• matchig > config > SecurityConfiguration.class
• config 파일은 config 패키지에 작성

• 프로젝트 상세 페이지를 위한 TEST REST API 구현

• User 도메인에 invest_time, social_url 컬럼 추가.

  • 바뀐 User 도메인에 대한 테스트 코드 작성.
    
    

• Project 도메인에 status 컬럼 수정 및 social_url 컬럼 추가.

  • 바뀐 Project 도메인에 대한 테스트 코드 작성.
    
    

• UserProject 조인 테이블에 status 컬럼 수정.

• 변경된 도메인에 대한 DTO들도 필드 추가 및 수정

  • 바뀐 DTO에 대한 테스트 코드 작성.

• User 생성 Post 및 Project 생성 Post 작성시에 유효성 검증 처리도 같이 진행.

• User를 생성할 때 UserSkill 도 같이 생성하도록 진행.

• Project를 생성할 때 Tag와 ProjectTag 도 같이 생성하도록 작성.

• Project 생성시에 현재 작성하는 User 기준으로 UserProject도 생성하도록 작성.

• API 추가 생성에 대한 README.md의 REST API 명세 최신화.

• 도메인 내에 컬럼들 중 상수형 데이터로 처리되는 컬럼이 있을 경우, enum으로 처리.

• 이후 상수형 데이터로 처리되는 데이터가 있을 경우, enum을 이용해서 처리하는 것을 권장.

• 도메인 모델링

• 모델링에 따른 관계성 매핑.

• 사진과 같이 추가되었음. 추후 post 부분 구현할때 참고해주세요. 🙂
• p.s. 앞으로 박은빈 팀원이 페이지 변동사항이 생길 때마다 와이어프레임에 반영하기로 함.
• 바뀔때마다 슬랙에 oven.io 주소를 보낼 예정.

• 필터링이 걸렸을 때 각각 원하는 데이터를 출력.

• 필터가 지역 또는 포지션 둘 중 하나만 있는 상황에 따른 데이터 처리.

• 필터가 지역과 포지션 둘 다 있는 상황에 따른 데이터 처리.

projectDetail 부분 tags 프로퍼티가 빈상태로 날라옵니다.
혹시 따로 리소스단위로 구현할 계획이 있다면 구현하고나서 명세해주세요~

• DoneProject 생성 로직 작성.

• DoneProject 수정 로직 작성.

• DoneProject 삭제 로직 작성.

• REST API 명세 작성.

• 기존에 HttpServletResponse를 이용해서 혜더를 담아주는 로직에서 API 요청시 ResponseEntity로 반환해줄 때 Link 헤더와 Location 혜더를 담아서 보내주는 방식으로 변경.

• 로직 변경으로 인해 프로젝트 내 모든 API 로직에 변경된 방식으로 리팩토링을 반양해야함.

1. Project.class의 변경 사항

• start_date, end_date, deadline 컬럼 삭제
• status Enum Data 수정 (모집중, 진행중, 완료)

2. DoneProject 도메인 생성

• 해당 도메인은 완료된 프로젝트의 정보와 외부에서 진행한 프로젝트를 커버하기 위함
• 명세에 따른 생성

3. Tag 도메인 생성

• 프로젝트 별 필요 기술을 태깅하고, 태깅 별 검색을 제공하기 위함
• 프로젝트와 M:N 관계성으로 매핑됨

4. UserSkill 도메인 생성

• 각 유저들이 보유한 스킬 정보를 담는 도메인

5. UsedrSkill 도메인 생성

• DoneProject에 기술 될 스킬 정보를 담는 도메인

해당 요청은 회원가입시에도 쓰이는 요청이라 토큰 없이도 가져올 수 있도록 해야할거 같아요~

jwt 토큰에서 로그인 유저 정보를 가져오려고 하는데,
email 뿐만아니라 idx와 nickname도 함께 담겨있어야 할거 같습니다.

그리고 유저 정보가 담겨있는 payload 부분의 키가 모호한데 사진과 같이 가능한지도 궁금합니다~

• UserProject 해당 테이블은 조인테이블로 table idx column 삭제
• PK, FK (user_idx, project_idx) 로 변경해야함

• REST API URL 매핑을 위한 명세 작성.

• 명세를 통하여 보다 명확한 API 사용을 위함.

• 내림차순 의존성 추가
• 해당 의존성이 사용되는 목적 기술
• 관련된 의존성은 모아서 관리
• 현재 사용되지 않는 의존성은 따로 관리

• 도메인 및 도메인들의 DTO에 대한 테스트 코드 작성.

• 생성 및 제약에 대한 테스트 코드 작성.

• gradle/wrapper/gradle-wrapper.properties 누락

• DTO class들에 idx 필드 추가
• DTO class들에 명세에 따른 제약추가
• ex) @NotBlack, @Length, @Email

MultipartFile을 이용한 File Upload, Downlaod 구현

• 파일 업로드, 다운로드 컨트롤러 구현
• 파일 관련 도메인을 생성하는 것에 대한 논의 필요
• 파일명 처리(중복 불가)

서버를 꺼놨거나 포트가 바꼈나요?
리스트 요청이랑 로그인 요청때 아래처럼 에러가 뜨네요

p.s. 추석 다 보내고 고치기

• User의 스킬들을 전부 가져올 수 있는 /api/userskills api 생성

• Project의 태그들을 전부 가져올 수 있는 /api/tags api 생성

• DoneProject의 스킬들을 전부 가져올 수 있는 /api/usedskills api 생성

• API 추가 생성에 대한 README.md의 REST API 명세 최신화.

• 프로젝트 도메인 Status Put 로직 작성.

  • 모집중 -> 진행중.

  • 진행중 -> 진행완료.

• REST API 명세 작성.

• project 도메인 생성자 누락
• comment 도메인 생성자 누락

POST 요청을 하니 400 에러가 뜨는데, 내가 보내야하는 데이터 포맷 양식 작성해줄수있나?

  const result = {
        content: "fewaf",
        designerRecruits: 1,
        developerRecruits: 0,
        etcRecruits: 2,
        location: "부산",
        marketerRecruits: 2,
        plannerRecruits: 1,
        socialUrl: "",
        summary: "awer",
        tags: [],
        title: "feafaew"
      };

이렇게 보내는중

• 각 도메인별 DB 명세에 따른 제약추가

• 예시

  @Column(length=255, nullable=false)
  private String content;

현재 생성된 token에 user와 관계성을 매핑하여 DB에 저장하고 상태를 나타내는 컬럼을 이용해 해당 토큰의 유효성을 검증하는데, 굳이 모든 토큰들을 DB에 저장할 필요가 없다고 생각됨, 관련 자료를 찾아본 결과 유저가 직접 로그아웃 요청을 한 토큰들에 대해서 블랙리스트를 만들어 관리하는 방법, 엑세스 토큰과 리프레쉬 토큰 두 가지를 이용해 관리 하는 방법 등을 찾을 수 있었음. 관련자료

• API 요청시에 JWT Token에 있는 payload 정보가 필요한 경우, JWT Token을 파싱하게 되는데, 그 로직이 중복되는 부분이 많아서 변경이 필요함.

테스트를 위한 데이터셋 추가

• 프로젝트 지원 하기 로직 작성 POST

• 프로젝트 지원자 관리 로직 작성 PUT

  • 대기중 -> 매칭.

  • 대기중 -> 거절.

• 지원 취소 로직 작성. DELETE

• REST API 명세 추가.

• Project 도메인 put 요청 처리 로직 작성.

• Project 도메인 delete 요청 처리 로직 작성.

1, ProfileDTO 수정 사항

• 각 유저가 보유하고 있는 스킬 태그 필드 추가

2. ProjectsDTO 수정 사항

• 각 프로젝트 별 필요 기술 태그를 의미하는 태그 필드 추가

3. ProjectDTO 수정 사항

• 상기 내용과 동일

4. DoneProjectDTO 생성

• 완료된 프로젝트와 외부 프로젝트를 처리하는 DoneProjectDTO 생성

• 변경된 내용들을 적용하기 위한 서비스 로직 변경 필요.

• Login 및 Logout 기능 구현.

• Spring Security 를 이용하여 구현.

• JWT 형식으로 로그인 시에 클라이언트에게 유효한 토큰을 반환해줄 것.

• 로그아웃시 토큰의 유효성을 만료시키도록 구현할 것.

• JWT Token을 서버단에서 관리하도록 구현할 것.

• project_detail page에 해당 프로젝트와 관련된 인원들에 대한 데이터를 포함하도록 변경

  • ex) api/project/1/members -> 1번 프로젝트에 연관된 인원들에 대한 정보

현재 리스트로 받아오는 프로젝트 데이터( project 리스트, doneProject 리스트)의 기술 태그 부분이
json 데이터에 프로퍼티로 있는게 아니라 link 태그로 한번 더 요청을 해서 가져오게 되어있는데, 한번 더 요청하게 하는게 아니라 tags: [ {idx: 1, text: 스프링}, { idx: 2, text: Node.js}, ... ] 라는 프로퍼티로 나오게 하는게 더 좋을거 같다.

단일 프로젝트에서 리소스 단위로 요청하는건 얼마 안되서 괜찮치만, 리스트로 보여주는 페이지들은
4개가 있다면 4번요청, 10개씩 불러온다면 10번 요청, 20개씩 불러온다면 20번 요청을 해야되니까 페이지는 한개인데 서버로 요청도 너무 많이가고 버그로 이어질 수 도 있을거 같아서 제안합니다

• Comment 도메인 post 요청 처리 로직 작성.

• Comment 도메인 put 요청 처리 로직 작성.

• Comment 도메인 delete 요청 처리 로직 작성.

• 추가로 작성된 로직에 대해서 api 명세 추가.

• Project-detail 테이블 추가.

• Project 요약 컬럼 추가

• DB 모델링 명세 수정.