Skip to content

Conversation

@toothlessdev
Copy link
Contributor

@toothlessdev toothlessdev commented Dec 15, 2025

1️⃣ PR 타입 (PR Type)

  • ✨ feat: 새로운 기능 추가
  • 🐛 fix: 버그 수정
  • ♻️ refactor: 코드 리팩토링
  • 📝 docs: 문서 수정
  • 🎨 style: 코드 스타일 변경 (포맷팅, 세미콜론 추가 등)
  • chore: 빌드 스크립트 수정, 패키지 매니저 설정 변경 등

2️⃣ 변경 사항 (Changes)

3️⃣ 관련 이슈 (Related Issues)

✅ 체크리스트 (Checklist)

  • base branch를 확인했나요?
  • self-review를 진행했나요?
  • 관련 이슈를 연결했나요?
  • 커밋 컨벤션을 준수했나요?
  • PR 제목 컨벤션을 준수했나요?

@toothlessdev toothlessdev added the 기능 기능 구현이 필요합니다. label Dec 15, 2025
Copy link
Contributor

@kimgho kimgho left a comment

Choose a reason for hiding this comment

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

zod는 따로 안 쓸 예정인가욥?

@toothlessdev
Copy link
Contributor Author

toothlessdev commented Dec 15, 2025

zod는 따로 안 쓸 예정인가욥?

서버 응답에 대한 ZOD Schema 가 필요할까요 ?
@kimgho @eunwoo-levi 어떻게 생각하시나요??

@eunwoo-levi
Copy link
Contributor

전체적으로 entities 별로 타입을 잡아둔 방향은 좋은 것 같습니다! 다만 현재 구현은 런타임 스키마가 아니라 TS 타입(interface) 정의라서 Schema 네이밍이 오해 소지가 있을 것 같습니다.

schemas/*Schema.tsmodel/types.ts 처럼 컨벤션을 통일하면 더 명확할 것 같은데 어떻게 생각하시나요??
그리고 user 엔티티만 schemas/ 폴더 없이 파일이 바로 있어 구조가 달라 다른 엔티티와 동일한 구조로 맞추면 좋을 것 같습니다!

export interface UserSchema {
id: number;
email: string;
role: "NORMAL" | "ADMIN";
Copy link
Contributor

Choose a reason for hiding this comment

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

roleUserRole 타입으로 분리하면 재사용성과 가독성이 좋아질 것 같습니다!

@kimgho kimgho merged commit 623043e into develop Dec 15, 2025
4 checks passed
@kimgho kimgho deleted the feature#19 branch December 15, 2025 13:08
@kimgho kimgho restored the feature#19 branch December 15, 2025 14:48
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.

[📌 Feature] RDB 테이블에 기반한 FSD 의 entities/*/schema 구성

4 participants