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(fe): add register code dialog, split register logic based on invitationCodeExists #1835

Merged
merged 2 commits into from
Jul 22, 2024

Conversation

B0XERCAT
Copy link
Member

@B0XERCAT B0XERCAT commented Jul 21, 2024

Description

Deregister 로직을 주석처리하고, 초대 코드 입력 모달을 구현했습니다.

Register.Code.Dialog.mp4

1. Upcoming Contest

Register Now 버튼이 비활성화됩니다.

2. 초대코드가 없는 Ongoing Contest

Register Now 버튼을 통해 바로 등록 가능합니다.

3. 초대코드가 있는 Ongoing Contest

Register Now 버튼을 통해 초대코드 입력 모달을 열고
모달에 올바른 초대코드를 입력하는 것으로 대회에 등록합니다.

참고사항

모달에 초대코드 입력할 때 경고 문구를
'Register Code must be a 6-digit number'로 설정했습니다.
피그마의 경우 'Incorrect'로 되어 있으나,
초대코드의 형식과 값 중 어느 것이 틀린 것인지 혼동 될 걸 우려해 수정했습니다.

추가 변경 사항 (7/22)

  • 기획 변경에 따라 Register Code -> Invitation Code로 수정했습니다
  • Admin Contest Create에서 초대코드를 입력하고 문제를 import 하러 페이지를 이동했을 때
    로컬 스토리지에 초대코드가 저장될 수 있도록 수정했습니다.
    (이전에 초대코드 관련 Admin 개발할 때 깜빡했었습니다...)

Closes #1811

Additional context


Before submitting the PR, please make sure you do the following

Copy link

vercel bot commented Jul 21, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
codedang ✅ Ready (Inspect) Visit Preview Jul 22, 2024 6:28am

Copy link
Contributor

@jihorobert jihorobert left a comment

Choose a reason for hiding this comment

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

리뷰 완료했습니다~

@B0XERCAT B0XERCAT merged commit 0b8a75b into main Jul 22, 2024
15 checks passed
@B0XERCAT B0XERCAT deleted the 1811-implement-contest-register-modal branch July 22, 2024 07:10
jihorobert pushed a commit that referenced this pull request Jul 28, 2024
…itationCodeExists (#1835)

* feat(fe): add register code dialog, split register logic based on invitationCodeExists

* chore: change register code to invitation code
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done ✔️
Development

Successfully merging this pull request may close these issues.

Contest 초대 코드 모달 구현
2 participants