Project 2 - 5일 차 (6) : 내가 push 한 회원가입 함수 : pull request 팀원 리뷰 + 수정본 ** 해야 함

username === undefined 는 !username 으로 축약

 

수정본

수정본

---

필수사항 및 선택사항 정보 받아온 거 저장하기 

수정본

수정본

그런데, 선택사항은 어디서 받아오지? 
key error는 필수사항에 대한 건데?  

--> insert into query문에서 
선택 + 필수 정보 모두 포함

만약, 유저가 선택 정보 기입을 안 했다면 'Undefined/ -'로 뜨겠지만, 
일단 쓸 경우를 대비해서 가져오긴 해야지 

 

---

[const error = new Error("KEY_ERROR");
error.statusCode = 400;
throw error;

해당 부분은 utils 폴더의 throwError(400, 'KEY_ERROR') 함수를 사용해서 축약될 수 있을 것 같습니당]

 

계속 반복되는 부분이라,

아마 이것을 활용하면 될 거 같다! 

 

수정본

아직...

---

[ !emailRegex.test(email)

해당 부분은 utils 폴더의 isValidData(emailRegex, email) 함수를 사용해서 축약될 수 있을 것 같습니당]

 

 

수정본

아직...

---

pr comment 에는 없지만, 내가 발견한 오류 1 

수정본

SELECT databaseId ->SELECT  id 

 

근데 select * FROM users WHERE email = '${email}' 
하면 안되는지? 
--> 최대한, * 은 안 쓰는 게 좋음. 쓰지 않을 데이터까지 다 가져오는 데이터 낭비가 되니까. 
-->  select id, email FROM users WHERE email =  '${email}' 

---

pr comment 에는 없지만, 내가 발견한 오류 2 

 

1차 프로젝트에도 막판에 통신할 때 깨달은 부분이였는데,

회원가입 시,

받아온 유저의 비밀번호를 

데이터베이스에 저장할 때,

비밀번호 해쉬화를 안 했다! 

 

수정본

 

---

이후 pull  과정 

 

 

깃허브 pr 이후 commit 

commit 해도 깃허브 메인 페이지에 나오지 않는구나. 

이렇게 comment에만 나오는구나 

 

(foundation test, project 1 에 걸쳐서, project 2도 하다보니 

이제 익혀지고 있다) 

 

궁금해서 찾아본 ㅎ