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.

Sign up for GitHub

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

Jump to bottom

[Refactor] 로그인 에러 페이지 반환 #151

Merged
merged 10 commits into from
Feb 2, 2025
Merged

[Refactor] 로그인 에러 페이지 반환 #151

merged 10 commits into from
Feb 2, 2025

Conversation

seokjun01
Copy link
Contributor

개요

가천대 계정으로 로그인 한 것이 아닐 경우 나타나는 뷰 로직을 추가하였습니다.
추가로, 프로필 정보 수정 시, 원하는 필드만 수정가능하게끔 코드 수정 하였습니다.
image
스크린샷 2025-02-01 오전 2 09 54

data 필드 중 ,닉네임 mbti중 바꾸고 싶은 필드의 값만 넣어주면 됩니다.
프로필을 바꾸고싶으면 data는 안넣어주시면 됩니다

PR 유형

어떤 변경 사항이 있나요?

  • 새로운 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제

PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • 커밋 메시지 컨벤션에 맞게 작성했습니다.
  • 변경 사항에 대한 테스트를 했습니다.(버그 수정/기능에 대한 테스트).

📣 To Reviewers

@seokjun01 seokjun01 added refactor 코드 리팩토링 석준 labels Jan 31, 2025
@seokjun01 seokjun01 requested review from dyk-im, jj0526 and ehs208 January 31, 2025 17:10
@seokjun01 seokjun01 self-assigned this Jan 31, 2025
ehs208
ehs208 reviewed Feb 1, 2025
View reviewed changes
Copy link
Member

@ehs208 ehs208 left a comment •
edited
Loading

Choose a reason for hiding this comment

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

코드상의 문제는 보이지 않아 코멘트로 남깁니다!
우선 이슈보시면 아시겠지만, 클라이언트에서 mbti값을 대문자가 아닌 소문자로 입력했을 때 발생하는 오류에 대해 처리해주셔야 될 것 같습니다!

또한 현재 뷰 파일이 .gitignore때문에 올라오지 않고있습니다.
figma에 있는 로그인 관련 에러페이지가 있긴한데,, 직접 구현.. 은 어려워보이니 프론트랑 협의후 프론트 페이지로 리다이렉트 해주는게 맞을 것 같습니다.

마지막으로 PR 컨벤션 준수 부탁드립니다!

dyk-im
dyk-im approved these changes Feb 1, 2025
View reviewed changes
Copy link
Contributor

@dyk-im dyk-im left a comment

Choose a reason for hiding this comment

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

예외 처리에 대한 코드 수정 모두 확인했습니다. 여러 케이스를 고려해 예외처리 해주신 부분과 도메인이 맞지 않는 경우 다른 페이지를 리다이렉트 해주는 부분 모두 이상 없이 구현된 것 같습니다!

jj0526
jj0526 approved these changes Feb 1, 2025
View reviewed changes
Copy link
Contributor

@jj0526 jj0526 left a comment

Choose a reason for hiding this comment

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

수고하셨어요!

src/main/java/com/example/eatmate/global/auth/login/oauth/OAuthLoginFailureHandler.java Outdated
CommonException commonException = (CommonException)cause;
if (commonException.getErrorCode() == ErrorCode.INVALID_EMAIL_DOMAIN) {
log.error("가천대 이메일이 아님: {}", exception.getMessage());
response.sendRedirect("/invalid-email.html");
Copy link
Contributor

Choose a reason for hiding this comment

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

가천대 이메일이 아닐 경우 따로 리다이렉트 해주는 부분 좋습니다!

@seokjun01 seokjun01 changed the title Refactor/#115 로그인 에러 페이지 반환 [Refactor] 로그인 에러 페이지 반환 Feb 2, 2025
@seokjun01 seokjun01 merged commit b453563 into dev Feb 2, 2025
2 checks passed
@seokjun01 seokjun01 deleted the refactor/#115-로그인-에러-페이지-반환 branch February 2, 2025 17:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dyk-im dyk-im dyk-im approved these changes

@jj0526 jj0526 jj0526 approved these changes

@ehs208 ehs208 ehs208 approved these changes

Assignees

@seokjun01 seokjun01

Labels
refactor 코드 리팩토링 석준
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[refactor] 로그인 에러 페이지 반환
4 participants
@seokjun01 @dyk-im @jj0526 @ehs208