KimGrang / prisma_project

0 stars 0 forks source link

Feedback #1

Open bewisesh91 opened 7 months ago

bewisesh91 commented 7 months ago

필수 요구사항


피드백

필수 요구 사항의 일부를 잘 구현하신 것 같습니다. 다만, 현재 API 명세서 URL && ERD URL 접속이 불가하여 일부 내용 확인이 불가해요. 아직 구현하지 않으신 것이거나 접속 오류라면 추후, 저에게 다시 말씀 주시면 감사할 것 같습니다.

일반적으로 코드 자체는 깔끔하게 잘 작성하시는 것 같아요. 대부분의 API가 이해하기가 쉽고, 분명합니다. 다만, 코드 스니펫을 보고 작성하셨다고 하였는데, 추후에는 동원님이 조금 더 직접 작성해보는 것에 힘을 주시면 좋을 것 같습니다.

또한, 깃 커밋 이력 역시 현재 한번에 모든 것을 다 업데이트한 것 같아요. 기능 단위별로 커밋을 작게 잘라서 관리하는 것도 연습해보면 좋을 것 같습니다.

개인 과제 고생하셨어요! ㅎㅎㅎ 다음 번엔 선택 요구 사항도 도전해아요 화이팅 👍

KimGrang commented 7 months ago

API명세서와 ERD 링크 수정했습니다. (링크[https://stone-artichoke-8a8.notion.site/Node-API-ERD-2987e13f920e411ca97979eaa05aa4ef])

이력서 정렬 기능이 작동하게 수정했습니다.(key, value에 따라 정렬)

추가로, 로그인 부분 clientId로 로그인 가능하게 수정했습니다. (선택 1

권한("normal","super")은 만들었지만, "super"권한자가 모든 이력서를 수정 가능하게 하는건 구현하지 못했습니다. (모든 사람이 모든 이력서를 수정할 수 있게 동작하고 있습니다.)(선택 2

swagger를 사용하는 것은 아직 시도중입니다.(2/6 시작)