Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: Memory -> Category 도메인명 변경 및 API URI 변경 #595 #608

Open
wants to merge 252 commits into
base: develop
Choose a base branch
from

Conversation

Junyoung-WON
Copy link
Contributor

@Junyoung-WON Junyoung-WON commented Jan 31, 2025

⭐️ Issue Number

🚩 Summary

  • 추억 ➡️ 카테고리로 도메인 명이 변경됨에 따라 프로젝트 내부의 Memory를 Category로 변경하했습니다.
  • 서버 API의 URI 변경 사항을 적용했습니다.
    • memory ➡️ category
    • moment ➡️ staccato
  • 사용하지 않는 도메인 모델 AccountInformationMyProfile을 제거했습니다.
  • Util의 URI ➡️ MultipartBody 변환 메서드(convertExcretaToFile)의 이름을 convertStaccatoUriToFile로 변경했습니다.

🙂 To Reviewer

  • moment 또는 memory가 어딘가에 남아있지 않은지 확인해주세요!
  • Comment API의 URI는 아직 moment로 남아있습니다! 백엔드 크루들에게 문의 후에 변경할게요!

📋 To Do

  • 빙티의 프로필 이미지 변경 기능 병합 후 작업 마저하기
  • v.1.2.2 배포

API 메서드의 파라미터명 변경
- memoryId -> categoryId
- memoryRequest -> categoryRequest
API 메서드명 변경
- ...Memory -> ...Category
- ...Memories -> ...Categories
CATEGORY_ID 상수 값 변경
- "/{categoryId}" -> "categoryId"

CATEGORY_PATH_WITH_ID 상수 값 변경
- "$CATEGORIES_PATH$CATEGORY_ID" -> "$CATEGORIES_PATH/{$CATEGORY_ID}"

CURRENT_DATE 상수 추가
- "currentDate"

Path Parameter, Query Value 상수화
- CATEGORY_ID
- CURRENT_DATE
메서드의 파라미터명 변경
- memoryId -> categoryId
- newMemory -> newCategory
메서드명 변경
- ...Memory -> ...Category
- getMemories -> getCategories
- memoryApiService -> categoryApiService
- MemoryRemoteDataSource -> CategoryRemoteDataSource
- memoryDataSource -> categoryDataSource
- memoryId -> categoryId
- newMemory -> newCategory
- MemoryDefaultRepository -> CategoryDefaultRepository
- memoryId -> categoryId
- memoryTitle -> categoryTitle
- memoryId -> categoryId
- memoryTitle -> categoryTitle
- MemoriesResponse -> CategoriesResponse
- MemoryCandidateResponse -> CategoryCandidateResponse
- MemoryCreationResponse -> CategoryCreationResponse
- memoryThumbnailUrl -> categoryThumbnailUrl
- memoryTitle -> categoryTitle
- memoryThumbnailUrl -> categoryThumbnailUrl
- memoryTitle -> categoryTitle
- MemoryRequest -> CategoryRequest
- memoryId -> categoryId
- memoryThumbnailUrl -> categoryThumbnailUrl
- memoryTitle -> categoryTitle
- memoryId -> categoryId
- memoryThumbnailUrl -> categoryThumbnailUrl
- memoryTitle -> categoryTitle
- MemoryResponse -> CategoryResponse
- MemoryStaccatoDto -> CategoryStaccatoDto
- MemoryMapper -> CategoryMapper
TimelineResponse.toCategoryCandidates() 메서드 내부 변수명 변경
- memories -> categories
API URL 변경에 따라, serial name의 moment를 staccato로 변경
- moments -> staccatos
API URL 변경에 따라, serial name의 moment를 staccato로 변경
- moment -> staccato
- momentImageUrl -> staccatoImageUrl
API URL 변경에 따라, serial name의 moment를 staccato로 변경
- momentImageUrls -> staccatoImageUrls
API URL 변경에 따라, serial name의 moment를 staccato로 변경
- momentId -> staccatoId
API URL 변경에 따라, serial name의 moment를 staccato로 변경
- momentId -> staccatoId
API URL 변경에 따라, serial name의 moment를 staccato로 변경
- momentLocationResponses -> staccatoLocationResponses
API URL 변경에 따라, serial name의 moment를 staccato로 변경
- momentId -> staccatoId
- momentImageUrls -> staccatoImageUrls
API URL 변경에 따라, serial name의 moment를 staccato로 변경
- momentImageUrls -> staccatoImageUrls
- momentImageUrls -> staccatoImageUrls
- momentImageUrls -> staccatoImageUrls
API URL의 변경에 따라, moment를 staccato로 변경
- moments -> staccatos
- momentId -> staccatoId

동반객체 활용
- STACCATO_ID = "staccatoId"
- STACCATO_PATH_WITH_ID = "$STACCATOS_PATH/{$STACCATO_ID}"
feature/#595 branch에 develop branch의 최신 변경 사항을 반영
- import 알파벳 순서에 따라 재정렬
- 들여쓰기 조정
- line당 글자수 초과한 코드 개행 처리
- 업데이트할 기분을 String이 아닌 도메인 모델 Feeling으로 변경
- StaccatoDefaultRepository에서 toFeelingRequest Mapper 메서드 활용
@Junyoung-WON Junyoung-WON added android We are android>< refactor 리팩토링 (변수 및 메서드 네이밍 변경) labels Jan 31, 2025
@Junyoung-WON Junyoung-WON added this to the sprint-9 milestone Jan 31, 2025
@Junyoung-WON Junyoung-WON requested review from s6m1n and hxeyexn January 31, 2025 09:54
@Junyoung-WON Junyoung-WON self-assigned this Jan 31, 2025
- MemoryCandidateFixture -> CategoryCandidateFixture
- Memory -> Category
- MemoryStaccato -> CategoryStaccato
Copy link
Contributor

@hxeyexn hxeyexn left a comment

Choose a reason for hiding this comment

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

도메인명 변경하시느라 정말 고생 많으셨습니다!!! memory를 category로 잘 바꿔주셨네요~
호두 덕분에 개발할 때 도메인명 때문에 혼동이 생기진 않겠네요~ 감사합니다! 🙇‍♂️

그나저나 커밋이 251개라니.. 역시 갓두🔥

URI에 남아있는 moment를 staccato로 변경
- 버저닝을 위해 comments 경로에 /v2 추가
- momentId -> staccatoId 로 변경
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
android We are android>< refactor 리팩토링 (변수 및 메서드 네이밍 변경)
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

refactor: 도메인명 변경 (Memory -> Category, API URI 도메인 변경)
2 participants