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

Closed
linirini opened this issue Jan 30, 2025 · 0 comments · Fixed by #607
Closed

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

linirini opened this issue Jan 30, 2025 · 0 comments · Fixed by #607
Assignees
@linirini
Labels
backend We are backend>< fix 버그 (버그 수정)
Milestone
sprint-9

Comments

@linirini
Copy link
Contributor

🤮 As Is (오마이갓 비상사태)

어떤 상황에서 발생한 버그인지 설명해주세요. (육하원칙이면 더 좋아요!)

스타카토 조회 API의 응답 값 맵핑에 오류가 있었습니다.
categoryId -> staccatoId
staccatoId -> categoryId

Image
Image

StaccatoDtoMapper

Image
Image

🤬 To Be

버그가 없었다면 어떻게 동작해야 하는지 설명해주세요.

categoryId -> staccatoId
staccatoId -> categoryId

😇 이때까지 끝낼게요!

버그 해결 예상 날짜를 작성해주세요. 신중하게 생각해요!

1/30에 바로 수정하겠습니다!

😵 참고 자료(선택)
@linirini linirini added the fix 버그 (버그 수정) label Jan 30, 2025
@linirini linirini self-assigned this Jan 30, 2025
@linirini linirini added the backend We are backend>< label Jan 30, 2025
@linirini linirini mentioned this issue Jan 30, 2025
Merged
@linirini linirini linked a pull request Jan 30, 2025 that will close this issue
fix: 스타카토 조회 API 응답값 매핑 오류 수정 #606 #607
Merged
linirini added a commit that referenced this issue Jan 30, 2025
@linirini
fix: StaccatoDetailResponse 응답값 매핑 오류 수정 #606 (#607)
5023ce7
@linirini linirini added this to the sprint-9 milestone Jan 31, 2025
@linirini linirini moved this to Done in 2024-staccato Jan 31, 2025
@linirini linirini closed this as completed by moving to Done in 2024-staccato Jan 31, 2025
s6m1n pushed a commit that referenced this issue Jan 31, 2025
@linirini @s6m1n
fix: StaccatoDetailResponse 응답값 매핑 오류 수정 #606 (#607)
ee371e3
Junyoung-WON pushed a commit that referenced this issue 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
Assignees

@linirini linirini

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

Successfully merging a pull request may close this issue.

fix: 스타카토 조회 API 응답값 매핑 오류 수정 #606 woowacourse-teams/2024-staccato
1 participant
@linirini