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.

Sign up for GitHub

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

Jump to bottom

fix: 스타카토 조회 API 응답값 매핑 오류 수정 #606 #607

Merged
merged 1 commit into from
Jan 30, 2025
Merged

fix: 스타카토 조회 API 응답값 매핑 오류 수정 #606 #607

merged 1 commit into from
Jan 30, 2025

Conversation

linirini
Copy link
Contributor

⭐️ Issue Number

🚩 Summary

  • 스타카토 조회 API 응답의 스타카토 ID와 카테고리 ID 응답값 매핑이 잘못되어 있어 수정했습니다.

🛠️ Technical Concerns

🙂 To Reviewer

  • StaccatoDtoMapper와 CategoryDtoMapper에 오류가 없는지 체크했습니다. 놓친 부분이 있을지도 모르니 더블 체크 부탁드립니다.
  • @hxeyexn @s6m1n @Junyoung-WON 조만간 바뀔 코드라서 테스트가 따로 없습니다. 체크하긴 했지만, 도메인명이 바뀐 API (Category, Staccato)에서 문제가 있는 것 같다면 백엔드 파트도 콜해주세용!

📋 To Do

@linirini linirini self-assigned this Jan 30, 2025
@linirini linirini added backend We are backend>< fix 버그 (버그 수정) labels Jan 30, 2025
@linirini linirini added this to the sprint-9 milestone Jan 30, 2025
@linirini linirini linked an issue Jan 30, 2025 that may be closed by this pull request
fix: 스타카토 조회 API 응답 값 매핑 오류 수정 #606
Closed
@github-actions GitHub Actions
Copy link

Test Results

 33 files   33 suites   6s ⏱️
238 tests 238 ✅ 0 💤 0 ❌
255 runs  255 ✅ 0 💤 0 ❌

Results for commit 3ef6bd3.

@github-actions GitHub Actions
Copy link

🌻Test Coverage Report

Overall Project 78.55% 🍏
Files changed 100% 🍏

File Coverage
StaccatoDtoMapper.java 96.3% 🍏

@linirini linirini changed the title fix: 스타카토 조회 API 응답값 매핑 오류 수정 fix: 스타카토 조회 API 응답값 매핑 오류 수정 #606 Jan 30, 2025
Junyoung-WON
Junyoung-WON approved these changes Jan 30, 2025
View reviewed changes
Copy link
Contributor

@Junyoung-WON Junyoung-WON left a comment

Choose a reason for hiding this comment

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

순서가 저렇게 바뀌어있었군요!ㅋㅋㅋㅋ
빠르게 수정해주셔서 감사드립니다! 🙇‍♂️

BurningFalls
BurningFalls approved these changes Jan 30, 2025
View reviewed changes
Copy link
Contributor

@BurningFalls BurningFalls left a comment

Choose a reason for hiding this comment

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

허걱쓰 제가 수정하는 과정에서 잘못했나보네요 ㅠㅠ
빠르게 대신 고쳐주셔서 감사합니다 리니!! 🙇 🙇 🙇

@linirini linirini merged commit 5023ce7 into develop Jan 30, 2025
2 checks passed
@linirini linirini deleted the feat/#606-fix-staccato-dto-mapper branch January 31, 2025 06:55
s6m1n pushed a commit that referenced this pull request Jan 31, 2025
@linirini @s6m1n
fix: StaccatoDetailResponse 응답값 매핑 오류 수정 #606 (#607)
ee371e3
Junyoung-WON pushed a commit that referenced this pull request Feb 6, 2025
@s6m1n @linirini
feat: 프로필 이미지 변경 및 타임라인 닉네임 표시 #538 (#603)
f3b1e15 
* chore: 사용되지 않는 interface 파일 삭제

* refactor: PhotoAttachFragment의 uriSelectedListener가 초기화 되지 않았을 때 예외 구체화

* refactor: formDataName 정의를 위한 String 상수를 FileUtils.kt로 이동

* feat: 마이 페이지에서 프로필 이미지 변경 기능 구현

* refactor: MyPageMenuHandler -> MyPageHandler로 이름 변경

* feat: 프로필 이미지 변경 실패 메시지 추가 및 예외 처리

* ui: 메인 화면 프로필 사진 테두리 디자인 수정

* refactor: 마이페이지에서 네트워크 요청 대신 캐시 메모리로 멤버 프로필 로드

* feat: SharedViewModel에 Hilt 적용 및 멤버 프로필 로직 추가

* feat: UserInfoPreferencesManager에 프로필 사진, 닉네임, 복구 코드 관련 로직 추가

* feat: 메인 화면 진입 시 멤버 프로필 불러오는 로직 추가

* feat: 메인에서 마이페이지 이동 시 ActivityResultLauncher 적용

* ui: 타임라인 타이틀 사용자 닉네임 표시

* refactor: 불필요한 데이터 바인딩 변수 제거

* refactor: ApiResult의 제네릭 타입 제약 제거

* refactor: MyPageRemoteDataSource 추상화 추가

* refactor: MyPageRemoteDataSource 추상화 적용

* refactor: MyPageLocalDataSource 추상화 추가

* refactor: UserInfoPreferencesManager가 MyPageLocalDataSource를 구현하도록 수정

* refactor: MyPageLocalDataSource 추상화 적용

* refactor: SharedPreferencesModule 추가

* refactor: MyPageDefaultRepository에 MyPageLocalDataSource 추가

* refactor:
메인 화면, 마이페이지 화면에 MyPageDefaultRepository 적용

* refactor: 프로퍼티 이름에서 멤버 변수를 뜻하는 m 키워드 제거

* refactor: 닉네임, 복구코드 관련 메서드 반환 타입 non-nullable로 수정

* refactor: 바인딩 어댑터 formatNickname 가독성 개선

* style: 컨벤션에 맞게 메서드 순서 변경

* delete: 불필요한 drawableRes 파일 삭제

* ui: 프로필 사진 수정 버튼 터치 영역 확대

* refactor: MyPageActivity 메서드 순서 변경

* fix: StaccatoDetailResponse 응답값 매핑 오류 수정 #606 (#607)

* fix: 오타 수정

* refactor: 바인딩 변수 네이밍 변경 menuHandler -> myPageHandler

* refactor: EMPTY_STRING를 MemberProfile 내부로 이동

---------

Co-authored-by: linirini <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@BurningFalls BurningFalls BurningFalls approved these changes

@Junyoung-WON Junyoung-WON Junyoung-WON approved these changes

@Ho-Tea Ho-Tea Awaiting requested review from Ho-Tea

Assignees

@linirini linirini

Labels
backend We are backend>< fix 버그 (버그 수정)
Projects
2024-staccato
Status: Done
Milestone
sprint-9
Development

Successfully merging this pull request may close these issues.

fix: 스타카토 조회 API 응답 값 매핑 오류 수정
3 participants
@linirini @BurningFalls @Junyoung-WON