[BE] feat: 카카오 소셜 로그인 (#17) - [merged]

[cheesecat47]

In GitLab by @jiyun3016 on Jan 30, 2024, 14:51

_Merges BE/Feat/17_kakaologin -> develop

로그인 및 토큰 발급 완료

[cheesecat47]

In GitLab by @jiyun3016 on Jan 30, 2024, 15:27

added 29 commits

• 4df8e26b...5b91d43c - 27 commits from branch develop
• ab3ff617 - Merge remote-tracking branch 'origin/develop' into BE/Feat/17_kakao_login
• 58595e78 - [BE] rename: merge 후 commit

Compare with previous version

[cheesecat47]

P4: API 명세서에는 없는 엔드포인트인 것 같은데, 추가된 내용일까요?

[cheesecat47]

P3: 코드와 주석이 큰 관계는 없어 보이는데, javadoc 등으로 한 데 모아줄 수 있을까요?

[cheesecat47]

P4: !6 에서는 ResponseEntity.status(HttpStatus.OK).body(...)와 같이 사용했던데, 둘 중 한 가지 방법으로 통일하는 것이 좋아보입니다.

[cheesecat47]

P4: API 명세서에는 없는 엔드포인트인 것 같은데, 추가된 내용일까요?

[cheesecat47]

P5: Builder 패턴을 사용한 이유가 있을까요?

[cheesecat47]

P5: 이 주석은 어떤 의미인가요?

[cheesecat47]

P2: stdout 사용은 지양 부탁합니다. log.debug가 좋을 듯 합니다.

[cheesecat47]

In GitLab by @jiyun3016 on Jan 30, 2024, 16:02

Commented on DEVoca-api/src/main/java/com/ssafy/devoca/user/controller/KakaoController.java line 19

추가하겠습니다.

[cheesecat47]

In GitLab by @jiyun3016 on Jan 30, 2024, 16:04

Commented on DEVoca-api/src/main/java/com/ssafy/devoca/user/controller/KakaoController.java line 19

코드 이해에 필요한 주석이라고 생각해서 적어두었지만 의견 참고하여 삭제하겠습니다.

[cheesecat47]

In GitLab by @jiyun3016 on Jan 30, 2024, 16:06

Commented on DEVoca-api/src/main/java/com/ssafy/devoca/user/service/KakaoService.java line 30

변수로 선언된 URI가 밑에서 주석에 쓰여진 URI 생성하는 곳에 사용됩니다. 참고하기 위해 적어두었습니다.

[cheesecat47]

In GitLab by @jiyun3016 on Jan 30, 2024, 17:55

added 11 commits

• 58595e78...a74b552c - 9 commits from branch develop
• 84c4d3d2 - [BE] refactor: log 추가, 주석, ResponsEntity 사용 포맷 변경
• 33356d7c - Merge remote-tracking branch 'origin/develop' into BE/Feat/17_kakao_login

Compare with previous version

[cheesecat47]

In GitLab by @amysia on Jan 30, 2024, 18:02

Commented on DEVoca-api/src/main/java/com/ssafy/devoca/user/controller/KakaoController.java line 20

P3: Error의 경우 처리 방식을 #10 등과 통일하는 게 좋지 않을까요??

[cheesecat47]

In GitLab by @jiyun3016 on Jan 30, 2024, 22:03

Commented on DEVoca-api/src/main/java/com/ssafy/devoca/user/controller/KakaoController.java line 20

앞으로 반영하도록 하겠습니다. :grinning: