Skip to content

Conversation

@lcs9317
Copy link
Collaborator

@lcs9317 lcs9317 commented Sep 22, 2025

제목(필수): [TYPE]: 제목 예) [FEAT]: 회원가입 기능 추가

제목 규칙 자세히 보기
  • 형식: [TYPE]: 제목
  • 제한: 50자 이내, 첫 글자 대문자, 명령문
  • TYPE: FEAT FIX REFACTOR COMMENT STYLE TEST CHORE INIT

무엇을 / 왜

  • 무엇(What):
  1. 베이스 노드를 생성한다
  2. 베이스 노드를 한꺼번에 라인에 넣고 오름차순 정렬 한다.
  3. API의 엔드포인트를 수정했음(중간부분은 수정 x)
  4. 정적 팩토리 매서드를 대신하는 Mapper를 생성
  • 왜(Why):
  1. 베이스 노드를 생성하고 그에 맞는 응답을 위해
  2. 베이스 라인을 생성해서 프론트에서 호출하여 유저와 연결 후 볼 수 있게 만들기 위해서
  3. API가 현재 적용 돼있는 상태로는 그 기능들을 모두 구현할 수가 없어서 수정함
  4. 매퍼를 생성하여 각각 DTO에서 관리하기보단 하나의 매퍼에서 관리하여 유지보수성을 높임

어떻게(요약) — 3줄 이내

  • 베이스, 결정, 결정중의 컨틀롤러들을 전부 분리해야 유지보수에 용이하여 분리하고 매퍼 인터페이스를 생성하여 글로벌에서 생성 후 각자 도메인에서 의존성을 가져와서 사용하는 식으로 만듬
    API를 수정하여 모든 기능을 온전히 수행할 수 있게 API의 종류를 늘리고 이름을 좀더 명확하게 변경함

영향 범위

  • API 변경
  • DB 마이그레이션
  • Breaking Change
  • 보안/권한 영향
  • 문서/가이드 업데이트 필요

체크리스트

  • 타입 라벨 부착 (FEAT/FIX/REFACTOR/COMMENT/STYLE/TEST/CHORE/INIT)
  • 로컬/CI 테스트 통과
  • 영향도 점검 완료
  • 주석/문서 반영(필요 시)

ToDo (선택)

  • 할 일 1
  • 할 일 2

스크린샷/증빙(선택)

이슈 연결 (자동)

Closes #15

Copy link
Collaborator

@johnbosco0414 johnbosco0414 left a comment

Choose a reason for hiding this comment

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

테스트코드 디스플레이네이밍 앞에 성공목적인지 실패목적인지만 붙여주면 좋을것같습니다.
ex. 성공 - 라인생성테스트
고생하셨습니다.

Copy link
Collaborator

@shihan00321 shihan00321 left a comment

Choose a reason for hiding this comment

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

확인했습니다.

@lcs9317 lcs9317 merged commit 009fb36 into main Sep 22, 2025
3 checks passed
@lcs9317 lcs9317 deleted the node/1 branch September 22, 2025 07:29
@lcs9317 lcs9317 added the FEAT 새로운 기능 추가 label Sep 22, 2025
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.

[FEAT]: 베이스 노드/라인 생성 및 조회와 Mapper 및 API수정

5 participants