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: 프로필 등록 완료 페이지 #1566

Merged
merged 8 commits into from
Sep 24, 2024
Merged

feat: 프로필 등록 완료 페이지 #1566

merged 8 commits into from
Sep 24, 2024

Conversation

simeunseo
Copy link
Member

@simeunseo simeunseo commented Sep 21, 2024

🤫 쉿, 나한테만 말해줘요. 이슈넘버

🧐 어떤 것을 변경했어요~?

  • 35기 여부에 따른 카드 뒷면 로고 분기처리 (35기면 AND SOPT 로고, 그 외에는 playground 로고)
  • 35기 여부에 따른 하단 버튼 섹션 분기처리 (35기면 홈으로가기 + 다짐메시지 진입점, 그 외에는 홈으로 가기만)
  • 다짐메시지 모달 연결
  • 다짐메시지 모달에 대해 로깅 추가 (profileUploadResolution)

🤔 그렇다면, 어떻게 구현했어요~?

맥락 설명드리자면, 원래 메이커스 4기 전에는 프로필 등록 완료 페이지가 존재하지 않았는데요
4기에서 처음으로 다짐메시지를 오픈하면서 진입점을 추가로 만들기 위해 이 프로필 등록 완료 페이지가 새로 만들어졌습니다.
그리고 다짐메시지 오픈 기간이 지난 후에, 이 페이지를 삭제하지 않고 계속 활용할 수 있도록
다짐메시지에 대한 CTA만 제거한 상태로 수정되어 운영이 되고 있었어요.

image

이렇게!

그리고 이번에 다짐메시지를 오픈하면 이렇게 바뀌는겁니다.

image

그런데 다짐메시지 오픈 기간동안에 신규 회원이 아닌 명예 회원도 가입할 가능성이 (희박하지만) 있기 때문에,
신규 회원과 명예 회원에 대한 뷰를 분기처리하게 되었습니다.
따라서 현재 회원이 35기인지 아닌지 여부를 확인하는 로직을 추가하게 되었습니다.

❤️‍🔥 당신이 생각하는 PR포인트, 내겐 매력포인트.

프로필 등록 완료 후에 진입할 수 있는 페이지라서 제대로된 테스트를 하려면 실제 프로필 등록 과정부터 거쳐야하는데요,
이 부분에 대해서는 백엔드쪽에 유저 정보를 삭제하고 회원가입부터 진행할 수 있도록 요청 후 QA를 진행해야 할 것 같아요.
그 전에는 임시로 /members/complete으로 직접 접속하시면 확인하실 수 있어요!

📸 스크린샷, 없으면 이것 참,, 섭섭한데요?

35기일 때

-35.mov
  • 모달 내 라이팅이 34기 버전으로 되어있는건 무시해주세요!

35기가 아닐때

-not35.mov

@simeunseo simeunseo self-assigned this Sep 21, 2024
Copy link

height bot commented Sep 21, 2024

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

Copy link

✨✨ 스토리북으로 확인하기 ✨✨

Copy link

🚀 프리뷰 배포 확인하기 🚀

https://c1549f7e.sopt-internal-dev.pages.dev

Copy link
Member

@pepperdad pepperdad left a comment

Choose a reason for hiding this comment

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

LGTM !
34기, 35기 계정으로 정상 동작하는 거 확인했습니다 ~!

@simeunseo simeunseo added this pull request to the merge queue Sep 24, 2024
Merged via the queue into main with commit 4edbec2 Sep 24, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: 프로필 등록 완료 페이지
2 participants