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/accept invite #53

Merged
merged 37 commits into from
Jul 11, 2024
Merged

Feat/accept invite #53

merged 37 commits into from
Jul 11, 2024

Conversation

11t518s
Copy link
Collaborator

@11t518s 11t518s commented Jul 5, 2024

배경

작업내용

초대 승인 로직

  1. 회원가입할 때 초대권을 생성하지 않고, 초대를 받았을 때 생성하도록 변경
  2. 아니 근데 internal에러는 왜 나지..........

yoon-junseo and others added 30 commits June 22, 2024 12:59
* client에서 사용할 때, trpc에서 invite router의 타입을 any로 잡혀서 상대경로로 변경
* feat: InviteInfo 테이블 추가

* feat: 초대권 생성 기능 구현

* feat: trpcModule, userModule에 inviteModule 추가

* feat: 회원가입시 초대권 생성 코드 추가

* chore: 코드 포맷팅 및 console.log 제거

* chore: 줄 띄어쓰기 추가

* feat: 내 초대권 조회 기능 추가

* feat: trpc router에 InviteController 연결

* feat: controller에서 사용하는 모듈들을 상대경로로 import 변경

* client에서 사용할 때, trpc에서 invite router의 타입을 any로 잡혀서 상대경로로 변경

* feat: 내 초대권 조회 기능 테스트 추가

* feat: code를 uid에서 랜덤 6자로 변경

* refactor: 불필요한 기본값 설정 제거

* refactor: 초대한 수 기록하는 변수에 기본값 설정 제거
schema상에서만 연결처럼 보이고 실제 db에서는 연결 안됨
실제 연결시키려면 schema.prisma에서 relationMode를 foreignKeys로 설정해야함
schema상에서만 연결처럼 보이고 실제 db에서는 연결 안됨
실제 연결시키려면 schema.prisma에서 relationMode를 foreignKeys로 설정해야함
@11t518s 11t518s requested a review from yoon-junseo as a code owner July 5, 2024 04:59
Copy link
Collaborator

@yoon-junseo yoon-junseo left a comment

Choose a reason for hiding this comment

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

지금 erd랑 안 맞는 부분 erd에 한번 업데이트해도 좋을듯 !

@@ -51,18 +51,8 @@ model InviteInfo {
tier String
}

enum Type {
enum User_type {
Copy link
Collaborator

Choose a reason for hiding this comment

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

별건 아닌데 enum은 UserType 요런식으로 쓰면 어떨까

Comment on lines 60 to 62
const ctxAsUser = ctx as User

return next({ ctx: ctxAsUser })
Copy link
Collaborator

Choose a reason for hiding this comment

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

상위에서 타입 단언을 해줘서 하위에서 사용할 때 안해도 되는거지 ??

Comment on lines +56 to +57
user User? @relation("UserInviteInfo", fields: [uid], references: [uid])
inviterTier InviteTier? @relation("UserInviteTier", fields: [tier], references: [tier])
Copy link
Collaborator

Choose a reason for hiding this comment

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

초대 정보에 user랑 inviteTier가 없을 수도 있었나 ??

11t518s and others added 2 commits July 8, 2024 01:58
schema상에서만 연결처럼 보이고 실제 db에서는 연결 안됨
실제 연결시키려면 schema.prisma에서 relationMode를 foreignKeys로 설정해야함
@11t518s 11t518s merged commit 1d29da1 into main Jul 11, 2024
@11t518s 11t518s deleted the feat/accept-invite branch July 11, 2024 13:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants