inkyu0103 / badminton-app

https://staging-mobae.xyz
1 stars 0 forks source link

feat: add user club model #156

Closed inkyu0103 closed 1 year ago

inkyu0103 commented 1 year ago

설명

E2E 테스트 시나리오

관련 이슈

Fix #155

변경사항

스크린샷

image

vercel[bot] commented 1 year ago

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

Name Status Preview Comments Updated (UTC)
badminton-app ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 1, 2023 7:47am
inkyu0103 commented 1 year ago

GET /clubs/1 이러면, 1번 모임에 대한 정보를 불러오는 내용 같은데...? (원래는 1번 유저가 속한 모임을 불러오려는 목적임)

GET /userclubs/1 이게 좀 더 나은가

관계 테이블(user_club)을 사용하는 모듈을 따로 만들어야 하는지? 언제 사용되는지?

고려할 것

수단 장점 단점
DB 유저의 상태를 실시간으로 알 수 있음 비용
JWT 비용 절감 ? 유저의 권한이나, 모임에 속한 여부가 바뀌더라도 곧바로 반영되지 않음

음... 일단 db를 조회하는 방식으로 해봅싀다

dadahee commented 1 year ago

GET /clubs/1 이러면, 1번 모임에 대한 정보를 불러오는 내용 같은데...? (원래는 1번 유저가 속한 모임을 불러오려는 목적임)

GET /userclubs/1 이게 좀 더 나은가

GET /users/{userId}/clubs는 어떤가요?

inkyu0103 commented 1 year ago

@dadahee

GET /users/{userId}/clubs는 어떤가요?

이게 좋을 것 같아요!

조금 사족을 붙여보자면, 이런 고민을 했던 이유는 큰 범위에서 볼 때 코드를 작성하는 스타일 차이가 아닐까 생각해요

상황 : user_club이라는 관계 테이블이 새로 생겼다.

user_club 테이블에 접근하는 방식은 2가지라고 생각했어요

생각해보니 굳이 연결테이블만 이용하는 모듈을 만들 필요가 있을까 싶네요. 행위 자체(?) 는 유저의 정보와 관련된 내용이니까요. 조언해줄게 있다면 또 댓글달아주세요 👍

inkyu0103 commented 1 year ago

모임 내용과 관련해서, 어떤 항목이 들어가야 할 지는 고민이 더 필요함

inkyu0103 commented 1 year ago

어... 테스트를 위한 seed 넣는 스크립트만 실행하면 죽는 이유는 뭐니?