🔗 :: (#871) 선생님 수동 등록 모집의뢰서 조회 api 추가

[Minnsuaa]

작업 내용 설명

• 선생님이 수동으로 등록한 기업의 모집의뢰서를 리스트로 조회합니다.

결과물(있으면)

체크리스트

• 어플리케이션 구동(혹은 테스트)시 오류는 없나요?
• DDL이 변경되었을 경우 flyway 마이그레이션 스크립트를 작성하였나요?
• 생성된 코드에 대한 테스트 코드가 작성 되었나요?

관련 이슈

• resolved 선생님 등록 기업 모집 의뢰서 리스트 조회 api #871

[4mjeo]

디자인에 보니 페이징도 구현해야 하지 않을까요?

[4mjeo]

깃모지 🔗로 수정 해주세요

[Minnsuaa]

디자인에 보니 페이징도 구현해야 하지 않을까요?

디자인상 페이징을 구현하는 것이 맞지만, 팀원과 상의한 결과 선생님이 수동으로 등록한 모집의뢰서의 개수가 많지 않을것으로 예상되어 디자인을 페이징 대신 스크롤 방식으로 변경하기로 하였습니다.

[geunoo]

일반 학생들은 개인컨택 모집의뢰서가 보이지 않게 되어야할 것 같은데, 그부분은 누가 하는건가요?

[geunoo]

vo 사용이 잘못된거 같은데요? 해당 vo는 companyid, name, logoUrl을 반환하는 용도인데, id부분에 recruitmentId를 넣으면 안될것같습니다.

[geunoo]

또 모집의뢰서 조회인데, 조회하는 모집의뢰서의 모집년도도 조건으로 걸려야되는거 아닌가요?

[Minnsuaa]

vo 사용이 잘못된거 같은데요? 해당 vo는 companyid, name, logoUrl을 반환하는 용도인데, id부분에 recruitmentId를 넣으면 안될것같습니다.

말씀하신 것처럼 CompanyVO가 CompanyId를 반환하도록 구현된것이 맞지만 나머지 필드가 조회하려던 값과 동일해서 별도의 VO를 만들지 않고 id만 바꾸어 기존 코드를 재활용 했었습니다. 하지만 CompanyVO를 사용하면 의미가 불분명할 수 있으니 변경하는것이 좋을까요?

[Minnsuaa]

또 모집의뢰서 조회인데, 조회하는 모집의뢰서의 모집년도도 조건으로 걸려야되는거 아닌가요?

제가 생각했을때 선배님께서 의견을 주신 이유는 수동으로 등록된 모집의뢰서들이 작성된 후 계속 남아있기 때문인것 같은데,
이에 대해 수동으로 등록된 모집의뢰서가 매년 남아있는지, 삭제되는지 여부를 팀원과 상의 후 남아있다면 구현하도록 하겠습니다.
만약 다른 이유가 있으셨다면 의견 주시면 감사하겠습니다!

[geunoo]

모집의뢰서는 매년 남아있어야 합니다!! 삭제된다는 선택지는 없습니다. 모집의뢰서가 삭제되면 해당 모집의뢰서에 종속되어있는 application이 삭제되고, 그럼 지원기록자체가 날라가는것이기때문에, 삭제되어선 안됩니다.

[geunoo]

vo를 재사용하는 것은 좋지만, 이런식으로 활용하면 의도가 불분명해질 수 있기때문에, 따로 만드는것이 좋을 것 같습니다.

[Minnsuaa]

모집의뢰서는 매년 남아있어야 합니다!! 삭제된다는 선택지는 없습니다. 모집의뢰서가 삭제되면 해당 모집의뢰서에 종속되어있는 application이 삭제되고, 그럼 지원기록자체가 날라가는것이기때문에, 삭제되어선 안됩니다.

답변 감사합니다! 모집년도로 조회하는 조건 추가하였습니다.

[Minnsuaa]

vo를 재사용하는 것은 좋지만, 이런식으로 활용하면 의도가 불분명해질 수 있기때문에, 따로 만드는것이 좋을 것 같습니다.

추가하도록 하겠습니다.

[geunoo]

근데 선생님 수동 등록 모집의뢰서를 따로 조회해야되나요? 기존 모집의뢰서 조회 페이지에서 필터링을 통해 조회하는 방법이 있는데, 제가 디자인을 못봐서 그런데 이 페이지가 달라서 api를 나누는건가요'?

[Minnsuaa]

일반 학생들은 개인컨택 모집의뢰서가 보이지 않게 되어야할 것 같은데, 그부분은 누가 하는건가요?

현재 자비스에서 학생들이 모집의뢰서를 조회할 경우 RecruitStatus가 RECRUITING인 모집의뢰서만 조회되는것 같아 선생님이 수동으로 회사를 등록하고 모집의뢰서가 생성될 경우 RecruitStatus가 MANUAL_ADD로 설정되어 학생들이 조회할 수 없다고 판단해 따로 구현하지 않았습니다.

[Minnsuaa]

근데 선생님 수동 등록 모집의뢰서를 따로 조회해야되나요? 기존 모집의뢰서 조회 페이지에서 필터링을 통해 조회하는 방법이 있는데, 제가 디자인을 못봐서 그런데 이 페이지가 달라서 api를 나누는건가요'?

이번에 추가되는 기능의 경우 위의 사진과 같이 조회가 되어 새로운 api를 개발했던 것입니다!

[geunoo]

근데 저 페이지는 개인컨택 모집의뢰서 화면이 아니라, 취업현황 관리 페이지 아닌가요? 저 페이지에는 취업한 학생의 기업이 모두 떠야되는거 아닌가요??

[Minnsuaa]

근데 저 페이지는 개인컨택 모집의뢰서 화면이 아니라, 취업현황 관리 페이지 아닌가요? 저 페이지에는 취업한 학생의 기업이 모두 떠야되는거 아닌가요??

해당 페이지는 취업현황 관리 페이지에서 선생님이 수동으로 등록한 회사의 모집의뢰서에 학생을 합격시키기 위한 모집의뢰서 조회 api로 해당 api를 통해 선생님이 수동으로 등록한 회사의 모집의뢰서와 회사정보를 조회해주고 이를 통해 학생을 합격처리할 수 있도록 하였습니다.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud