상호 평가 코드 리뷰

[odumag99]

안녕하세요! 5조 백엔드에 대한 상호 평가를 맡은 3조입니다.

코드가 워낙 깔끔하고 구조적으로 설계된 것 같아서 흠 잡을 곳이 딱히 없어 보이고, 저희도 보면서 많이 배워야겠다는 생각이 듭니다.
피드백의 목적으로 다음과 같은 점 말씀드리니 참고해주시면 감사하겠습니다.

1. wastory.app.user.views에서 'update_me'라는 엔드포인트명이 PATCH "/me"와 PATCH "/change_password"에서 중복되어 사용되고 있는데 혹시 이렇게 하신 이유가 있으신지 궁금합니다.

2. 현재 회원가입 과정에서 password가 평문으로 DB에 저장되고 있는 것으로 보입니다. 보안을 고려하여 암호화된 비밀번호를 DB에 저장하는 것이 좋을 것 같습니다.

3. pytest 등을 이용해 테스트 주도 개발을 시도하셨던 것으로 보입니다. 그 과정에서 API 구현의 정확성, 효율성 등의 측면에서 어떤 장단을 느끼셨는지 궁금합니다.

4. 지엽적인 부분이지만, category.views에서 GET /{category_id}가 특정 카테고리를 조회하는 API가 맞다면, 엔드포인트 명도 그에 맞게 수정하고, status_code도 200 Ok로 수정하는 것이 적절해 보입니다.

방대한 기능들을 모두 구현하시느라 고생 많으셨습니다!

[SooinPark1019]

리뷰해주셔서 정말 감사합니다!!

1 이거는 단순히 저희 실수...인 것 같습니다 ㅠㅠ 지적해주셔서 감사합니다.

3 테스트 주도 개발을 시도했다가 잘 안돼서...지금은 수동으로 테스트를 하고 있는데 확실히 API 구현의 정확성이나 효율성 측면에서 떨어지는 듯한 느낌을 받습니다. 다음에 이런 프로젝트를 진행하게 된다면 이 부분을 공부를 좀 해서 확실히 해놔야겠다는 생각을 했습니다.

2번과 4번은 새겨듣겠습니다! 덕분에 많이 배웠습니다. 남은 기간 동안 3조도 화이팅입니다!!