Skip to content

refactor(meetup): 모임 찾기, 만들기 관련 리팩토링#357

Merged
sohyun0 merged 5 commits into
developfrom
#355-refactor-meetup
May 8, 2026
Merged

refactor(meetup): 모임 찾기, 만들기 관련 리팩토링#357
sohyun0 merged 5 commits into
developfrom
#355-refactor-meetup

Conversation

@PollyGotACracker
Copy link
Copy Markdown
Collaborator

@PollyGotACracker PollyGotACracker commented May 2, 2026

🛠️ 설명 (Description)

모임 찾기, 만들기 관련 코드를 리팩토링하였습니다.

📄 설계 문서 (Design Document)

관련 설계 문서나 위키 페이지가 있다면 링크를 추가해주세요.(하위 항목 노션링크 첨부)

📝 변경 사항 요약 (Summary)

  • 모임 만들기 관련 중복 유효성 검사를 meetupDetail 유틸 함수로 통합 적용
    • validateDateTimeIsFuture
    • validateDateTimeOrder
    • validateMaxCapacity
    • validateCapacity 수정: 최소 인원은 고려하지 않도록 변경, validateMaxCapacity와 함께 활용
  • 모임 조회, 생성 API 호출에 throwApiError 적용
    • apis/meetings.ts
    • apis/meetingTypes.ts
    • apis/images.ts
    • features/meetup/apis.ts
  • 모임 찾기 관련 코드/파일 구조 정리
  • 모임 카드 표시 조건을 유틸에 관련 meetupDetail 컴포넌트 주석 추가

💁 변경 사항 이유 (Why)

  • 중복 코드를 제거하여 각 도메인 간 기능 불일치 해소

✅ 테스트 계획 (Test Plan)

  • api의 경우 관련 이슈에 테스트 파일 추가됨

🔗 관련 이슈 (Related Issues)

☑️ 체크리스트 (Checklist)

  • 코드가 프로젝트 코딩 컨벤션을 따릅니다.
  • 테스트 코드가 작성되었고, 통과했습니다.
  • 변경 사항에 대한 문서화가 완료되었습니다.
  • 필요한 경우, 다른 팀원에게 리뷰를 요청했습니다.
  • CI/CD 파이프라인이 성공했습니다.

👀 리뷰어를 위한 참고 사항 (Notes for Reviewers)

  • 상수 export는 features/meetup/constants로 모으고 features/meetup/utils는 그대로 re-export하였습니다.
    MIN_CONFIRMED_COUNT 값은 meetupDetail 에서 import 하였습니다.
// features/meetup/constants.ts
import { MIN_CONFIRMED_COUNT as MIN_CONFIRMED_COUNT_DEFAULT } from "@/features/meetupDetail/components/PersonnelContainer";

/** 모임 개설 확정 등에 쓰는 최소 인원(모집 정원 하한) */
export const MIN_CONFIRMED_COUNT = MIN_CONFIRMED_COUNT_DEFAULT;
/** 모임 이름 최대 길이 */
export const MAX_NAME_LENGTH = 20;
/** 주소 상세 최대 길이 */
export const MAX_ADDRESS_LENGTH = 50;
  • swagger 문서에서 개설 확정 알림이 아래와 같이 정의되고 있어 MIN_CONFIRMED_COUNT 값 검토가 필요합니다.
    현재 값은 3으로 작성되었습니다.
image

@PollyGotACracker PollyGotACracker self-assigned this May 2, 2026
@PollyGotACracker PollyGotACracker added the refactor 코드 리팩토링 / 코드 수정 label May 2, 2026
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request centralizes API error handling using a new throwApiError utility and refactors the meetup list logic by introducing helper functions for building requests and queries. It also updates validation logic for meetup creation and moves constants to a dedicated file. Feedback suggests decoupling constants from UI components to reduce coupling and renaming capacity validation functions to improve clarity, as the current naming is counter-intuitive for minimum value checks.

Comment thread features/meetup/constants.ts
Comment thread features/meetup/create/components/StepSchedule/index.tsx
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 8, 2026

🧷 Preview: https://reboot-73g6b5kp8-code-base.vercel.app
⏰ 배포 시각: 2026년 05월 08일 15시 38분

@sohyun0 sohyun0 merged commit ff50370 into develop May 8, 2026
1 check passed
@sohyun0 sohyun0 deleted the #355-refactor-meetup branch May 8, 2026 06:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor 코드 리팩토링 / 코드 수정

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[REFACTOR] 모임 찾기, 만들기 관련 리팩토링

2 participants