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

feat: 소셜 로그인 시 랜딩할 페이지 결정에 사용할 헤더 추가 #64

Merged
merged 4 commits into from
Feb 12, 2024

Conversation

uwoobeat
Copy link
Member

🌱 관련 이슈

📌 작업 내용 및 특이사항

  • 소셜 로그인 후 유저가 어디로 랜딩할지를 결정하는 헤더를 추가했습니다.
  • 헤더 이름은 Landing-Status 입니다.
  • 헤더는 TO_STUDENT_AUTHENTICATION / TO_REGISTRATION / TO_DASHBOARD 세 가지 상태를 가집니다.
    • 각각 재학생 인증 페이지 / 가입신청 페이지 / 대시보드 페이지를 의미합니다.
  • 결정 로직은 다음과 같습니다.
    • univStatusPENDING 이면 재학생 인증 페이지로 랜딩합니다.
    • 가입신청을 하지 않았다면 학번이 null입니다. 따라서 학번이 null이면 가입신청 페이지로 랜딩합니다.
    • 그 외의 경우에는 대시보드 페이지로 랜딩합니다.

📝 참고사항

📚 기타

@uwoobeat uwoobeat added the ✨ feature 새로운 기능 추가 및 수정 label Feb 12, 2024
@uwoobeat uwoobeat self-assigned this Feb 12, 2024
@uwoobeat uwoobeat requested a review from a team as a code owner February 12, 2024 09:57
Copy link
Member

@Sangwook02 Sangwook02 left a comment

Choose a reason for hiding this comment

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

한 가지 의문이 있는데 가볍게 확인해주세요
lgtm

Comment on lines +20 to +22
if (member.getStudentId() == null) {
return TO_REGISTRATION;
}
Copy link
Member

Choose a reason for hiding this comment

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

멤버에 이 부분 확인하는 용도의 다른 필드가 있는 것보다 이렇게 학번으로 확인하는게 좋을까요?

Copy link
Member Author

@uwoobeat uwoobeat Feb 12, 2024

Choose a reason for hiding this comment

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

가입신청 상태만 따로 관리하는 필드가 있으면 가능할 것 같은데...

저희가 가입신청 상태를 멤버역할과 통합해서 관리하고 있고,
이때 가입신청서를 작성했든 작성하지 않았든 같은 GUEST라서 구별이 어려운 것 같아요
그래서 가입신청서 작성할 때 필드값 업데이트 치니까 필드값 null 여부로 판단한거긴 한데... 좀 마음에 안들긴 하죠

가입신청상태와 멤버상태를 분리할 수 있으면 좋긴 하겠네요. 더 명확하기도 하고요
일단 급하니까 임시로 필드값 널 여부로 처리하는 게 맞을 것 같아요~

Copy link
Member

Choose a reason for hiding this comment

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

여유가 있으면 분리하는게 좋을 것 같긴한데
일단은 이렇게 가고 이슈 새로 만들어서 나중에 여유가 생기면 처리해도 좋을 것 같습니다.

내부 구현이라 프론트 쪽에 영향이 있는 것도 아니니까요

@uwoobeat uwoobeat merged commit a13d3f4 into develop Feb 12, 2024
1 check passed
@uwoobeat uwoobeat deleted the feature/61-landing-header branch February 12, 2024 10:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ feature 새로운 기능 추가 및 수정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

✨ 소셜 로그인 시 랜딩할 페이지 결정에 사용할 헤더 추가
2 participants