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

Merged
merged 256 commits into from
Feb 6, 2025

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 배포

메서드의 파라미터명 변경
- 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
- memoryId -> categoryId
- memoryTitle -> categoryTitle
- memoryId -> categoryId
- memoryTitle -> categoryTitle
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
@Junyoung-WON Junyoung-WON linked an issue Jan 31, 2025 that may be closed by this pull request
2 tasks
- 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로 잘 바꿔주셨네요~
호두 덕분에 개발할 때 도메인명 때문에 혼동이 생기진 않겠네요~ 감사합니다! 🙇‍♂️

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

URI에 남아있는 moment를 staccato로 변경
- 버저닝을 위해 comments 경로에 /v2 추가
- momentId -> staccatoId 로 변경
GET, POST 요청에만 v2 적용하도록 변경
- 버저닝 URI인 COMMENTS_URI_V2
다른 ApiService에서 사용하는 변수명과 통일
- URI -> PATH
@Junyoung-WON Junyoung-WON mentioned this pull request Feb 5, 2025
12 tasks
Copy link
Member

@s6m1n s6m1n left a comment

Choose a reason for hiding this comment

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

Commits 254...
File changed 129...
What the Hell?

너무 수고하셨습니다 호두!!!

- develop branch의 변경 사항을 해당 기능 branch(feature/#595)에 반영
- convertMemoryUriToFile -> convertCategoryUriToFile
@Junyoung-WON Junyoung-WON merged commit f8e3aeb into develop Feb 6, 2025
1 check passed
@hxeyexn hxeyexn deleted the feature/#595-change-memory-to-category branch February 6, 2025 09:53
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: Done
Development

Successfully merging this pull request may close these issues.

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