Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Chore : 로그인 응답 수정 #25

Merged
merged 2 commits into from
Jul 10, 2024
Merged

Conversation

jsilver01
Copy link
Member

@jsilver01 jsilver01 commented Jul 10, 2024

요약 (Summary)

  • 깃허브 소셜로그인할 때 기존 회원의 profile 정보를 추가로 넣었습니다.
  • 명세서도 수정하였습니다. 수정된 명세서

🔑 변경 사항 (Key Changes)

  • LoginResponse 수정 : 기존의 response 에서 profile 만 추가로 반환하도록 하였습니다.
  • LoginService 수정 : 기존의 서비스문은 4개의 쿼리가 날라가서 확인해보았더니 checkmember 메소드를 사용하지 않으니 3개의 쿼리로 줄어든 것을 확인하였습니다. 이렇게 수정해도 큰 문제가 생길 것 같지 않아 수정해보았습니다. 만약 데이터베이스에 없다면 명세서처럼 모든 값이 Null 로 처리됩니다.

📝 리뷰 요구사항 (To Reviewers)

  • 회원이 아닐 때에 명세서 대로 반환이 되는지
  • 회원일 경우 데이터베이스에 저장된 profile 과 토큰이 잘 생성되는지
  • 바뀐 서비스 로직에 문제가 없는지

확인 방법

❗️application-local.yml 을 로컬 환경에 맞춰주세요.

헤더에 토큰은 필요하지 않습니다. 로그인작업을 해주세요.
스크린샷 2024-07-10 오후 3 27 01

회원이 아닐경우 위와 같이 반환됩니다.

쿼리문을 실행시켜주세요.

use linkode;

INSERT INTO avatar (created_at, modified_at, avatar_img, status) VALUES
('2024-06-30 12:00:00.000000', '2024-06-30 12:30:00.000000', 'img1.png', 'ACTIVE');
INSERT INTO avatar (created_at, modified_at, avatar_img, status) VALUES
('2024-06-30 12:00:00.000000', '2024-06-30 12:30:00.000000', 'img2.png', 'ACTIVE');
INSERT INTO color (created_at, modified_at, hex_code, status) VALUES
('2024-06-30 12:00:00.000000', '2024-06-30 12:30:00.000000', '헥스코드1', 'ACTIVE');
INSERT INTO color (created_at, modified_at, hex_code, status) VALUES
('2024-06-30 12:00:00.000000', '2024-06-30 12:30:00.000000', '헥스코드2', 'ACTIVE');

INSERT INTO member (created_at, modified_at, avatar_id, github_id, nickname, color_id,status) VALUES
('2024-06-30 12:00:00.000000', '2024-06-30 12:30:00.000000', 1, 'jsilver01','초콜릿','1', 'ACTIVE');
INSERT INTO member (created_at, modified_at, avatar_id, github_id, nickname, color_id,status) VALUES
('2024-06-30 12:00:00.000000', '2024-06-30 12:30:00.000000', 2, 'Mouon', '두바이','2', 'ACTIVE');

다시 로그인해주세요.
스크린샷 2024-07-10 오후 3 27 42

다음과 같이 반환되면 정상입니다!

@Mouon
Copy link
Member

Mouon commented Jul 10, 2024

리뷰 유구사항대로 수정된것 모두 확인했습니다! 수고하셨습니다!

if(memberStatus){
LoginResponse.Profile profile = null;
if(member.isPresent()){
Member member1 = member.get();
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

순수 질문드립니다!

Member member1 = member.get(); 

이 아니라
위에 생성한 member로 getter를 사용하면 문제가 될까요?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

아 Optional이라 안되네요! 머지하겠습니다~!

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ㅋㅋㅋㅋ넵!!!

@Mouon Mouon merged commit c9bd0aa into develop Jul 10, 2024
@Mouon Mouon deleted the LINKODE-69-BE-로그인json수정 branch July 10, 2024 08:05
@Mouon Mouon changed the title Feat : 로그인 json 수정 Feat : 로그인 응답 수정 Jul 10, 2024
@jsilver01 jsilver01 changed the title Feat : 로그인 응답 수정 Chore : 로그인 응답 수정 Jul 15, 2024
@Mouon Mouon added the Chore New feature or request label Sep 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Chore New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants