-
Notifications
You must be signed in to change notification settings - Fork 3
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: 프로필 등록 완료 페이지 #1566
feat: 프로필 등록 완료 페이지 #1566
Conversation
|
🚀 프리뷰 배포 확인하기 🚀 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM !
34기, 35기 계정으로 정상 동작하는 거 확인했습니다 ~!
🤫 쉿, 나한테만 말해줘요. 이슈넘버
🧐 어떤 것을 변경했어요~?
profileUploadResolution
)🤔 그렇다면, 어떻게 구현했어요~?
맥락 설명드리자면, 원래 메이커스 4기 전에는 프로필 등록 완료 페이지가 존재하지 않았는데요
4기에서 처음으로 다짐메시지를 오픈하면서 진입점을 추가로 만들기 위해 이 프로필 등록 완료 페이지가 새로 만들어졌습니다.
그리고 다짐메시지 오픈 기간이 지난 후에, 이 페이지를 삭제하지 않고 계속 활용할 수 있도록
다짐메시지에 대한 CTA만 제거한 상태로 수정되어 운영이 되고 있었어요.
이렇게!
그리고 이번에 다짐메시지를 오픈하면 이렇게 바뀌는겁니다.
그런데 다짐메시지 오픈 기간동안에 신규 회원이 아닌 명예 회원도 가입할 가능성이 (희박하지만) 있기 때문에,
신규 회원과 명예 회원에 대한 뷰를 분기처리하게 되었습니다.
따라서 현재 회원이 35기인지 아닌지 여부를 확인하는 로직을 추가하게 되었습니다.
❤️🔥 당신이 생각하는 PR포인트, 내겐 매력포인트.
프로필 등록 완료 후에 진입할 수 있는 페이지라서 제대로된 테스트를 하려면 실제 프로필 등록 과정부터 거쳐야하는데요,
이 부분에 대해서는 백엔드쪽에 유저 정보를 삭제하고 회원가입부터 진행할 수 있도록 요청 후 QA를 진행해야 할 것 같아요.
그 전에는 임시로
/members/complete
으로 직접 접속하시면 확인하실 수 있어요!📸 스크린샷, 없으면 이것 참,, 섭섭한데요?
35기일 때
-35.mov
35기가 아닐때
-not35.mov