Skip to content

refactor: [TA-178] API 함수 네이밍 컨벤션 통일 (CRUD 동사 기준)#96

Merged
seorinn merged 5 commits into
developfrom
TA-178-api-naming-convention
May 11, 2026
Merged

refactor: [TA-178] API 함수 네이밍 컨벤션 통일 (CRUD 동사 기준)#96
seorinn merged 5 commits into
developfrom
TA-178-api-naming-convention

Conversation

@seorinn

@seorinn seorinn commented May 11, 2026

Copy link
Copy Markdown
Collaborator

요약

src/api/ 폴더의 API 함수 이름을 REST CRUD 동사 기준으로 통일합니다.

변경 내용

photographer.ts

이전 이후
photographerRegistration createPhotographer
savePhotographer createSavedPhotographer
putPhotographerPage updatePhotographerPage
modifyPhotographerProfile updatePhotographerProfile

user.ts

이전 이후
userRegistration createUser
modifyProfile updateProfile

promotion.ts

이전 이후
postPromotion createPromotion
savePromotion createSavedPromotion
putPromotion updatePromotion

호출부 일괄 업데이트

  • src/components/PhotographerInfo/index.tsx
  • src/pages/EditPhotographerPage/index.tsx
  • src/pages/EditPhotographerProfile/index.tsx
  • src/pages/EditUserProfile/index.tsx
  • src/pages/EventDetail/BottomBar/index.tsx
  • src/pages/NewEvent/useEventForm.ts
  • src/pages/SignUp/index.tsx

컨벤션 기준

  • 조회: get* (현행 유지)
  • 생성: create*
  • 수정: update*
  • 삭제: delete* (현행 유지)

테스트 방법

  • TypeScript 빌드 통과 확인 (npx tsc --noEmit)
  • 회원가입 흐름 정상 동작 확인
  • 작가 프로필/페이지 수정 정상 동작 확인
  • 유저 프로필 수정 정상 동작 확인
  • 이벤트 생성/수정 정상 동작 확인
  • 작가/이벤트 저장 기능 정상 동작 확인

관련 이슈

TA-178

@vercel

vercel Bot commented May 11, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
damaba Ready Ready Preview, Comment May 11, 2026 6:55am

@seorinn seorinn marked this pull request as ready for review May 11, 2026 06:55
@seorinn seorinn merged commit 6645627 into develop May 11, 2026
3 checks passed
@seorinn seorinn deleted the TA-178-api-naming-convention branch May 11, 2026 06:56
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.

1 participant