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: 유저 회원가입 플로우 (TEMP_USER -> USER) #10

Merged
merged 9 commits into from
Mar 20, 2024
Merged

Conversation

h-beeen
Copy link
Member

@h-beeen h-beeen commented Mar 20, 2024

PULL REQUEST

🎋 Issue Ticket

🔑 주요 작업사항

  • TEMP_USER ROLE의 유저의 이름, 생일, 성별, 기저질환&특이사항, 백곰아이디를 입력받아 USER ROLE로 변경합니다.

  • USER ROLE의 AccessToken을 새로 발급하고, 이를 바탕으로 추후 메인페이지 이후 플로우의 API 구현

  • 현재 백신 조회 내역을 반영하지 않은, 순수 유저에게 입력받은 정보만을 담는 API입니다.

  • Request

POST /api/v1/member/signup HTTP/1.1
Host: https://api-dev.vacgom.co.kr
Content-Type: application/json
Authorization: Bearer *******
Cookie: JSESSIONID=4E5091ED3C56B2DB15B70EED9B6B3B30
Content-Length: 194

{
    "name": "변해빈",
    "birthday" : "1999-11-07",
    "sex" : "MALE",
    "healthConditions" : ["HEMATOPOIETIC_STEM_CELL_TRANSPLANTATION", "CHRONIC_LIVER_DISEASE"],
    "vacgomId": "h_beeen"
}
  • Response
{
    "member": {
        "memberId": "876ebfe2-2cf5-4a95-90fe-dc0544f48bd9",
        "role": "ROLE_USER"
    },
    "token": {
        "accessToken": "*******"
    }
}
{
    DIABETES(1, "당뇨병"),
    CHRONIC_CARDIOVASCULAR_DISEASE(2, "만성 심혈관질환"),
    CHRONIC_PULMONARY_DISEASE(3, "만성 폐질환"),
    CHRONIC_RENAL_DISEASE(4, "만성 신질환"),
    CHRONIC_LIVER_DISEASE(5, "만성 간질환"),
    SOLID_TUMOR_UNDERGOING_ANTINEOPLASTIC_THERAPY(6, "항암치료중인 고형암"),
    IMMUNOSUPPRESSIVE_AGENTS_EXCLUDING_TRANSPLANT(7, "이식 이외 면역 억제제 사용"),
    HEMATOPOIETIC_STEM_CELL_TRANSPLANTATION(8, "조혈모"),
    CELL_TRANSPLANTATION(9, "세포이식"),
    SICKLE_CELL_DISEASE(10, "무비증"),
    HIV_INFECTION_CD4_ABOVE_200(11, "HIV 감염:CD4>=200/mm3"),
    HIV_INFECTION_CD4_BELOW_200(11, "HIV 감염:CD4<200/mm3"),
    PREGNANCY(12, "\임신"),
    MEDICAL_WORKER(13, "의료기관 종사자"),
    ORGAN_TRANSPLANTATION(14, "장기 이식 경험")
}

🏞 (Optional) 참고 자료

  • Postman <백곰 회원가입> 참고

(중요) 서브모듈이 수정되었나요?

  • 기존 커밋 :
  • 변경 커밋 :
  • 변경 사항 :
  • [] 해당 서브모듈 변경사항이 PR에 잘 반영되었나요?

꼭 확인해 주세요!!

@h-beeen h-beeen self-assigned this Mar 20, 2024
@h-beeen h-beeen added the ✨ Feat 새로운 기능을 추가하는 경우 label Mar 20, 2024
@h-beeen h-beeen removed their assignment Mar 20, 2024
@h-beeen h-beeen added this to the Member Domain milestone Mar 20, 2024
@osohyun0224
Copy link
Member

확인했습니다!! 해빈님 고생많으셨습니다 !!

import jakarta.persistence.Embeddable

@Embeddable
class Nickname(
Copy link
Collaborator

Choose a reason for hiding this comment

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

이렇게 객체에 제약 거는거 좋아하는건 또 어케 하시고 ..
근데 이거 코틀린에서도 자바처럼 생성자를 만들 수 있거든여 그걸 활용하는게 좋을것 같습니다.

member.updateNickname(nickname)
member.updateMemberDetails(memberDetails)
member.updateRole(Role.ROLE_USER)

Copy link
Collaborator

Choose a reason for hiding this comment

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

여기서 나중엔 백신접종내역 추가되는ㄱ게 맞나요?

Copy link
Collaborator

@HyungJu HyungJu left a comment

Choose a reason for hiding this comment

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

넘넘 수고하셨습니다 ..
저희 일정상 플로우만 확인하고 Approve 합니다 !

감사합니다

@h-beeen h-beeen merged commit 9f15e2d into develop Mar 20, 2024
1 check passed
@h-beeen h-beeen deleted the feat/#9 branch March 20, 2024 12:51
@h-beeen h-beeen self-assigned this Mar 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feat 새로운 기능을 추가하는 경우
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants