LV3 코드리뷰

[9sanha]

전체적으로 주석이 많아서 너무 좋아요! 잘 하고 계십니다 👍

// 로그인 API 
router.post('/login', async(req, res)=>{
    const {nickname, password} = req.body;

    // 닉네임이 일치하는 유저를 찾는다.
    const user = await Users.findOne({ where: { nickname } });

    /* REVIEW: error message를 따로 핸들링하지 않는다면 위 조건문에 password까지 포함 시켜서 조회할 수 있겠네요! */
    //  1. 닉네임에 일치하는 유저가 존재하지 않거나
    //  2. 유저를 찾았지만, 유저의 비밀번호와, 입력한 비밀번호가 다를때,
    if(!user || user.password !== password) {
        res.status(400).json({ errorMessage: "닉네임 또는 패스워드를 확인해주세요" })
        return;
    }

추가적으로 Posts 테이블에 nickname을 넣으신 이유가 궁금하네요!

또한 config에 rds정보가 있는데 실제 계정 정보는 조심조심 해주세요!

[Munaso]

Posts 테이블에 nickname 을 넣은 이유는 게시글 조회 API 에서 nickname 도 반환하라고 과제 요구사항이 되어 있어서예요.

사실 저도 Posts 테이블의 nickname 컬럼은 빼고 싶어요 . 중복컬럼이니까요. nickname 컬럼을 빼고도 Users 테이블의 nickname 을 가져와서 반환할 수도 있을까요?

[9sanha]

네 ERD 이슈에 같이 첨부해주세요 :)

[Munaso]

일단 Posts 테이블의 nickname 컬럼 제거는 했습니다. 그 대신에 Users 테이블 참조해서 nickname 가져오는 작업도 했구요.

ERD 이슈에 같이 첨부하라는 말씀이 어떤.. 뜻일까요? 이해를 못했어요.

[9sanha]

DB ERD 설계하신 부분을 이슈나 해당 repository의 readme file에 추가해주세요 :)